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 "code_challenge_methods_supported" to well known response #503

Merged
merged 3 commits into from
Jul 5, 2023

Conversation

sascha-eisenmann
Copy link
Contributor

Hi, I am currently trying to use the mock server with the Okta JavaScript SDK. The client currently throws an error because the property "code_challenge_methods_supported" is missing in the .well-known response.
If I add the property, the mock server can be used to mock Okta requests, which is really cool for local development purposes.

@sascha-eisenmann sascha-eisenmann requested a review from a team as a code owner July 3, 2023 15:20
@sascha-eisenmann sascha-eisenmann force-pushed the master branch 2 times, most recently from 35dec0f to 2769b4f Compare July 4, 2023 05:51
@ybelMekk
Copy link
Contributor

ybelMekk commented Jul 4, 2023

Whoo. Nice additions. Im going to look into it later today.

@ybelMekk ybelMekk self-assigned this Jul 4, 2023
@ybelMekk ybelMekk self-requested a review July 4, 2023 08:18
README.md Show resolved Hide resolved
@ybelMekk
Copy link
Contributor

ybelMekk commented Jul 5, 2023

@jksolbakken or @tronghn anything to add before i merge this?

@tronghn
Copy link
Contributor

tronghn commented Jul 5, 2023

Would it make more sense to implement actual PKCE-support rather than declaring support in the metadata (but not actually support it)?

Wait nevermind, we do support PKCE already 😄

@ybelMekk ybelMekk merged commit dbc12d8 into navikt:master Jul 5, 2023
@tommytroen tommytroen added the enhancement New feature or request label Jul 6, 2023
@tommytroen
Copy link
Collaborator

@sascha-eisenmann your changes have been released in 0.5.10. Thanks for your contribution 🥇

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

Successfully merging this pull request may close these issues.

5 participants