Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Support PKCE on OIDC Clients #170

Merged
merged 4 commits into from
Oct 30, 2019
Merged

Conversation

PSanetra
Copy link
Contributor

This PR adds support for the advanced OIDC client setting Proof Key for Code Exchange Code Challenge Method on Keycloak >= 7.0.0

The setting is also applied on Keycloak versions < 7.0.0, but is ignored on those versions. (At least on Keycloak 6.0.1)

Fixes #169

@mrparkers
Copy link
Contributor

Thanks for the PR!

Could you run make fmt and push up the changes?

@PSanetra
Copy link
Contributor Author

@mrparkers I have pushed your suggested changes.

Copy link
Contributor

@mrparkers mrparkers left a comment

Choose a reason for hiding this comment

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

Looks great! I pushed some changes to your branch to add docs and bump the Keycloak version so this feature can be seen locally.

Thanks again for the PR!

@mrparkers mrparkers merged commit d119e7a into keycloak:master Oct 30, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support PKCE switch in Advanced Settings of openID Client
2 participants