-
Notifications
You must be signed in to change notification settings - Fork 20
Add Managed Identity support #181
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
Comments
Hi Sean Any news on this? I will be happy to help, but I guess I need a bit more information about how to do this :) |
@CasperWSchmidt, no I did not have a chance to look into that as I was busy with looking into #205 to remove the dependency on the Storage emulator, #198 which is a breaking API change and will require a new major release, and #201 which will be similar to #198 but not even ready yet. For this plugin to function, Managed Identity would only be needed for the Storage v11 SDK as that's the only thing that requires permissions to execute. For Storage SDK v11, you can already do that by passing a variant of |
Keep in mind that it would need not just the code change for the internals but also APIs review, documentation extension as Storage v11 and v12 are not the same. I'm working on things a little here and there but just didn't have enough time to wrap it up. I'm always happy to accept PRs 😃 |
Managed Identity for Storage Blobs v12 SDKs.
For Storage v11 SDK (attachment plugin v6),
StorageCredentials
withTokenCredentials
can be passed into configuration to authenticate using Managed Identity.The text was updated successfully, but these errors were encountered: