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

Fix issue #206 #245

Closed
wants to merge 1 commit into from
Closed

Fix issue #206 #245

wants to merge 1 commit into from

Conversation

AndreasBilke
Copy link

Use the same logic for confidential client authentication as in token
request with authorization grant.

List of common tasks a pull request require complete

  • Changelog entry is added or the pull request don't alter library's functionality

Use the same logic for confidential client authentication as in token
request with authorization grant.
@rosinisi
Copy link

rosinisi commented Apr 20, 2021

I found this very same issue that applies also to refreshToken. I might do a similar pull request for that method.

Edit: found already a pull request that concern this: https://github.com/jumbojett/OpenID-Connect-PHP/pull/215/files

@azmeuk
Copy link
Collaborator

azmeuk commented Nov 21, 2021

Fixed by #271. Sorry I did not see this patch before, as it is older I think this is the one that should have been merged.

@azmeuk azmeuk closed this Nov 21, 2021
@AndreasBilke AndreasBilke deleted the issue-206 branch November 22, 2021 07:51
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.

3 participants