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.

Sign up for GitHub

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

Jump to bottom

Issue #1213 fix (RFC 9207) #1327

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Issue #1213 fix (RFC 9207) #1327

wants to merge 2 commits into from

Conversation

buchatsky
Copy link

The fix for issue #1213 is implemented:

  • added AuthConfig.authorizationResponseIssParameterSupported (defaults to true)
  • added OidcDiscoveryDoc.authorization_response_iss_parameter_supported (takes precedence over AuthConfig)
  • added iss param validation in OAuth Authorization Response for Authorization Code flow and Implicit flow

@mahmut-gundogdu
Copy link
Contributor

I am waiting for merged/accepted.

@letonin
Copy link

letonin commented Jan 23, 2024

we are also waiting for a fix

@jicedtea
Copy link

Waiting for this fix

@mgimpactbuying
Copy link

Also waiting for this fix

@kcinay055679
Copy link

same

@L-X-T L-X-T force-pushed the master branch 4 times, most recently from a54ea92 to 078415b Compare December 2, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@buchatsky @mahmut-gundogdu @letonin @jicedtea @mgimpactbuying @kcinay055679