Skip to content

Conversation

@aaronpk
Copy link

@aaronpk aaronpk commented Feb 6, 2024

enable with pkce: true when creating the strategy.

Closes #83 and #56

enable with `pkce: true` when creating the strategy
@kamronbatman
Copy link

Great job, thank you!!

@darknightdev
Copy link

Great job! @kamronbatman We need this feature as well, can we please merge and publish? Thanks

@kamronbatman
Copy link

Waiting on @jaredhanson to validate, add tests, and determine if there are any breaking changes. It sounds silly but many people rely on this. If I get time next week I'll work with him on tests.

@MattPaymentus
Copy link

I tested this further. The call back on the backend doesn't pass the code challenge with it so it errors out.

@aaronpk
Copy link
Author

aaronpk commented Feb 26, 2024

@MattPaymentus not sure what you mean, I have this code working against an authorization server that requires PKCE.

@quichef
Copy link

quichef commented Jun 13, 2024

Hello, we'd be interested on my side with this PR

@Hugo-Dlg
Copy link

Hi @jaredhanson and @kamronbatman, any update on it ? Thanks

@VVlasy
Copy link

VVlasy commented Dec 9, 2024

Is the project maintainer unavailable?

@jackofsometrade
Copy link

Community is still waiting on support for PKCE support, FWIW...

@mdelnegro-lennd
Copy link

Any update?

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.

Issue with Authorization Code Flow

9 participants