Skip to content
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

azidentity 1.3.0 to support workload identitiy #188

Merged
merged 2 commits into from
May 23, 2023

Conversation

tareksha
Copy link
Contributor

azidentity 1.3.0 adds support for authentication via federated web identity, which in turns allows workload identity ( trust-based auth via service account token without supplying explicit credentials )

@tareksha tareksha force-pushed the azidentity_1_3_0 branch 2 times, most recently from 99d0c7c to d05da80 Compare May 11, 2023 20:51
@ywk253100
Copy link
Contributor

@tareksha Thanks for the PR!
Could you fix the CI failure?

Signed-off-by: Tareq Sharafy <[email protected]>
@tareksha
Copy link
Contributor Author

@ywk253100 i mod tidy'ed , the ci requires approval to run again

@codecov-commenter
Copy link

Codecov Report

Merging #188 (afd5853) into main (b5bc8ba) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #188   +/-   ##
=======================================
  Coverage   13.96%   13.96%           
=======================================
  Files           4        4           
  Lines         673      673           
=======================================
  Hits           94       94           
  Misses        576      576           
  Partials        3        3           

@qiuming-best qiuming-best merged commit c5714e2 into vmware-tanzu:main May 23, 2023
@tareksha tareksha deleted the azidentity_1_3_0 branch May 23, 2023 08:58
@Sh4kE
Copy link

Sh4kE commented May 23, 2023

Don't you think the docs should have been updated accordingly?

@ywk253100
Copy link
Contributor

@Sh4kE Will work on that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants