-
Notifications
You must be signed in to change notification settings - Fork 56
SDK
- Support MSI authentication in Azure Container App and App Service like environments
#1093
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
Conversation
… header in HTTP request
Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement Learn more about why HashiCorp requires a CLA and what the CLA includes Have you signed the CLA already but the status is still pending? Recheck it. |
This is awesome for the last months we had to use a proxy to rewrite the endpoint versions. Great work. |
This has been signed, not sure why its not updating. |
@vermacodes |
SDK
- Support MSI authentication in Azure Container App and App Service like environments
Any update on when this PR is scheduled to go live? |
@vermacodes, any updates on getting this merged to the main branch? |
I am not sure; it was never picked for review. May be @stephybun or @jackofallops knows. |
Any update on when this PR is scheduled to go live? |
Update: We're following up with the Azure IMDS service team on this to ensure that any change we make will actually be supported in all regions / clouds. Please bear with us. |
Thanks for the update. Could you share an approximate timeline for going live with the feature? Our application depends on it. Also, could you explain why the API version isn't configurable via an environment variable? |
Out of curiosity, did Azure IMDS end up responding with this as fully available? This appears to be exactly what is defined in their official Azure Container App documentation for MSIs https://learn.microsoft.com/en-us/azure/container-apps/managed-identity?tabs=cli%2Chttp#connect-to-azure-services-in-app-code |
@jackofallops, do you have any updates on this? |
Hi all - Thanks for the patience on this while we worked with the IMDS team (and other service teams) to get to a path forward. I'm closing this PR for an alternative implementation that avoids adding another hard-coded value, and allows for flexibility for API versions to change and for other special cases that don't use either of the current default api version, or the customised API version in use by Container Apps. The new PR will be open shortly and will expose a new provider configuration property (specifically |
Fixes #1054
Pull Request: Enhanced Support for Azure App Service Like Environments
This pull request updates
managed_identity_authorizer.go
to improve support for Azure App Service like environments. Key changes include:MSI_ENDPOINT
andMSI_SECRET
environment variables to detect App Service like environments.MSI_ENDPOINT
as the default instead of the well-known endpoint.2019-08-01
API version, which requires theIDENTITY_HEADER
in the HTTP request.These changes ensure better compatibility with Azure's managed identity in App Service like environments.