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

Added Organization to OAuth2. #7435

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

Conversation

CallumJohnson
Copy link

Closes: #7209

This is an optional parameter on OAuth2 endpoints for 'Business Users' in OAuth2 applications.
image
This required affiliation with an organization, therefore requires this option to be specified on the request.

@CLAassistant
Copy link

CLAassistant commented May 20, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@jackkav jackkav left a comment

Choose a reason for hiding this comment

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

I wonder if it might be better to add a custom values table rather than a new hard coded key, I'll discuss with the team, thanks for your contribution!

@CallumJohnson
Copy link
Author

Thanks for your quick response; a custom values table would be great for the OAuth2 flow.

@jackkav
Copy link
Contributor

jackkav commented May 21, 2024

I saw in postman they have three phases with custom query params, would some like that satisfy your use case? Or am I misunderstanding something?
image

@CallumJohnson
Copy link
Author

Absolutely, we're currently using the Auth Requests section to provide this context to our OAuth2 flow. Adding a table like this would be very powerful!

Thanks again for your response.

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.

None yet

3 participants