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

Add token request parameters for authorization code flow #192

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

Conversation

coberlin
Copy link

The access token request needs the code if we're using code flow. For authorization code flow, grant_type and redirect_uri are also required. See https://docs.duendesoftware.com/identityserver/v7/reference/endpoints/token/

To run tests, pin the minitest version per a conflict with mocha, as noted here: freerange/mocha#614
But do not leave pinned as the gem fails to install under some rubies that previously succeeded.

For example:
net-imap-0.5.0 requires ruby version >= 3.1.0, which is incompatible with the current version, ruby 2.6.8p0 (jruby 9.3.7.0)

The access token request needs the code if we're using code flow.
For authorization code flow, grant_type and redirect_uri are also required.
See https://docs.duendesoftware.com/identityserver/v7/reference/endpoints/token/

To run tests, pin the minitest version per a conflict with mocha, as noted here:
freerange/mocha#614
But do not leave pinned as the gem fails to install under some rubies that
previously succeeded.

For example:
net-imap-0.5.0 requires ruby version >= 3.1.0, which is incompatible with the
current version, ruby 2.6.8p0 (jruby 9.3.7.0)
@coberlin coberlin force-pushed the add-extra-token-request-params branch from 107f9e3 to 42df4f4 Compare November 1, 2024 18:19
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.

1 participant