Update quickstart sample app to use EntraID authentication - #1117
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates quickstart sample applications across multiple languages to use EntraID (Entra ID) authentication instead of connection string-based authentication for Azure App Configuration.
Key Changes:
- Replaced connection string authentication with DefaultAzureCredential for EntraID authentication
- Updated environment variable from
AZURE_APPCONFIG_CONNECTION_STRINGtoAZURE_APPCONFIGURATION_ENDPOINT - Added required identity/authentication libraries across all language examples
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| examples/QuickStart/Python/QuickStartApp/QuickStartApp.py | Updated to use DefaultAzureCredential with endpoint-based authentication, added azure.identity import |
| examples/QuickStart/JavaScript/QuickStartApp/QuickStartApp.js | Updated to use DefaultAzureCredential with endpoint-based authentication, added @azure/identity import |
| examples/QuickStart/JavaScript/QuickStartApp/package.json | Added @azure/identity dependency for authentication support |
| examples/QuickStart/JavaSpring/QuickStart/src/main/resources/bootstrap.properties | Changed from connection-string to endpoint property for EntraID authentication |
| examples/QuickStart/JavaSpring/QuickStart/pom.xml | Added azure-identity-spring dependency for authentication support |
| examples/QuickStart/DotNet/QuickStartApp/Program.cs | Updated to use DefaultAzureCredential with endpoint-based authentication, includes proper error handling |
| examples/QuickStart/DotNet/QuickStartApp/QuickStartApp.csproj | Added Azure.Identity package reference with version 1.14.0 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import os | ||
|
|
||
| connection_string = os.environ.get("AZURE_APPCONFIG_CONNECTION_STRING") | ||
| endpoint = os.environ.get("AZURE_APPCONFIGURATION_ENDPOINT") |
There was a problem hiding this comment.
Unlike the .NET example which includes explicit error handling for a missing environment variable, this code will silently pass None to the load function if AZURE_APPCONFIGURATION_ENDPOINT is not set. This could result in unclear error messages for users following the quickstart. Consider adding validation similar to the .NET example to provide a clear error message when the environment variable is not set.
Co-authored-by: Zhenlan Wang <zhenlwa@microsoft.com>
Co-authored-by: Zhenlan Wang <zhenlwa@microsoft.com>
…github.com/Azure/AppConfiguration into user/maryanne/update_quick_start_apps_auth
Zhenlan Wang (zhenlan)
left a comment
Zhenlan Wang (zhenlan)
left a comment
There was a problem hiding this comment.
Approving the PR but please don't merge it. We should coordinate this PR with the updated portal release and minimize the time they are inconsistent.
|
Zhenlan Wang (@zhenlan) Maryanne Gichohi (@MaryanneNjeri) Jiayi has told me the release has finished in Azure Cloud and will finish in all other clouds tomorrow. Seems we should be good to merge. |
|
Sounds good |
This PR
Updates our quick start sample apps to use EntraID auth instead of connection string