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

[provider] Fix OIDC support #2086

Closed
wants to merge 1 commit into from
Closed

[provider] Fix OIDC support #2086

wants to merge 1 commit into from

Conversation

pgavlin
Copy link
Member

@pgavlin pgavlin commented Nov 21, 2022

Add parameters to the client configuration used by the provider so that the provider supports OIDC authentication.

Add parameters to the client configuration used by the provider
so that the provider supports OIDC authentication.
@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Looking good! No API changes found.

@jaxxstorm
Copy link

Fixes #1324

@stefanhenseler
Copy link

@pgavlin awesome! Just the issue I started running into a couple of days ago. Any Idea when this will be released?

Copy link
Member

@danielrbradley danielrbradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This depends on moving the core authentication library from ADAL to MSAL. @thomas11 has made good progress in this area but there's still more investment needed to get it over the line yet.

@danielrbradley
Copy link
Member

Feel free to close this @pgavlin and we'll re-integrate this work once the core auth libray has been upgraded

@illusional
Copy link

Any luck on this front, would be awesome to use MSI w/ workload identity federation support with Pulumi!

@thomas11
Copy link
Contributor

Any luck on this front, would be awesome to use MSI w/ workload identity federation support with Pulumi!

Hi @illusional, there are some complications to work through, but the good news is we're working on it.

@thomas11 thomas11 added the resolution/duplicate This issue is a duplicate of another issue label Mar 31, 2023
@thomas11
Copy link
Contributor

superseded by #2320

@thomas11 thomas11 closed this Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution/duplicate This issue is a duplicate of another issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants