-
Notifications
You must be signed in to change notification settings - Fork 102
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
Add JWT token authentication option #161
Comments
I was looking for a way to pass the token obtained from Azure AD for authentication instead of username and password. Is there any plan or roadmap to support this? I was planning on using this together with Managed Service Identity and the Microsoft.Azure.Services.AppAuthentication library... |
Fixing this in #189 |
Hi, |
Yes, MultiShardConnection support is also needed and will be done. Thanks :) |
@jaredmoo |
MultiShardConnection fixed by #201 @jaredmoo any chance to get this merged and get hands on a pre-release soonish (alternatively release)? We are removing SQL Credentials from an App and the Elastic Scale Client is the only thing currently blocking us. Or are there currently any workarounds for passing AccessToken in version 2.3.0? |
I merged it :) sorry for the delay with the release, I have been running into challenges with our signing process that are taking a long time to work through, that's the only thing blocking release. I suggest you could build from master branch yourself for now as a workaround. |
Any updates on when this will be released? |
@jaredmoo - Are there any plans to release these updates? We'd like to implement and would prefer to do it via an official release path. Any info would be helpful. Thanks |
2.4.0 released to nuget.org: https://www.nuget.org/packages/Microsoft.Azure.SqlDatabase.ElasticScale.Client/2.4.0 Release notes: https://github.com/Azure/elastic-db-tools/releases/tag/v2.4.0 Let me know if you hit any issues |
This option exists in Azure SQL, and should unconditionally be part of this library
The text was updated successfully, but these errors were encountered: