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

bug: remove mandatory client secret variable #3573

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Dan6erbond
Copy link

Closes #3572

Description

Removes client secret validation to trigger client credentials flow when it isn't set.

Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

Additional Information

None.

@amk-dev
Copy link
Contributor

amk-dev commented Dec 12, 2023

@Dan6erbond

which would then trigger the client credentials flow for providers such as Azure AD.

Can you give me steps to reproduce your configuration in Azure AD or any other service that supports a client credential flow without a client secret.

@StormicK
Copy link

Is it possible to help out here? The PR seems quite dead and I need the oauth2 pkce flow without a client secret urgently.

Or do you have an idea how to get around this issue

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.

[bug]: Allow Client Secret for OAuth2 Authentication
3 participants