-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some errors found in scenario: frontend + backend + mongodb #4526
base: main
Are you sure you want to change the base?
Fix some errors found in scenario: frontend + backend + mongodb #4526
Conversation
{{- if .DbCosmosMongo}} | ||
cosmos | ||
{{- end}} | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this, the cosmos may created after ACA, it will return error like "can not get mongodb-url using managed identity xxx". This error is not caused of authorization, it's caused by the secret not written to keyvault yet.
'PUT' | ||
'POST' | ||
'DELETE' | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this, the frontend app can not do write / delete operation.
@@ -343,7 +354,7 @@ module {{bicepName .Name}} 'br/public:avm/res/app/container-app:0.8.0' = { | |||
{{- range $i, $e := .Frontend.Backends}} | |||
{ | |||
name: '{{upper .Name}}_BASE_URL' | |||
value: 'https://{{.Name}}.internal.${containerAppsEnvironment.outputs.defaultDomain}' | |||
value: 'https://{{.Name}}.${containerAppsEnvironment.outputs.defaultDomain}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The front end can not access the URL with internal.
inside. Delete internal.
will make it work well.
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash:
pwsh:
WindowsPowerShell install
MSI install
Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
Fix some errors found in scenario: frontend + backend + mongodb