-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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
Enhance clarity for workaround with unsupported Connection String Types #122464
base: main
Are you sure you want to change the base?
Conversation
This commit expands the workaround note for configuring connection strings in Azure App Service to increase visibility and understanding of the necessary workaround for connection string types that are not handled by `EnvironmentVariablesConfigurationProvider`. Specifically, it clarifies that the workaround to use the "Custom" type applies not only to PostgreSQL but also to other services like Notification Hub, Service Bus, Event Hub, Document Db, and Redis Cache.
@neoscie : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit d7940fc: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
#label:"aq-pr-triaged" |
Learn Build status updates of commit 3f82e7c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
This commit expands the workaround for connection string types that are not properly handled by the
EnvironmentVariablesConfigurationProvider
, as detailed in the already referenced issue. It clarifies that the "Custom" connection string type should be used not only for PostgreSQL but also for Notification Hub, Service Bus, Event Hub, Document Db and Redis Cache.Additionally, this update completes the list of connection string types currently available in Azure App Service, along with their respective environment variable prefixes, ensuring comprehensive guidance for users.