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 previous authorized token information to getNewToken() #1108

Closed
wants to merge 5 commits into from

Conversation

Richard87
Copy link

Solves #1103 and #672 and possibly #885

Ping: @Sephster
Does this look good? I'm pretty new to creating pull-requests!
(I used PHPStorms "Rebase onto branch... 9.0.0-WIP", and renamed my branch... Hope that solves the issue!)

christiaan and others added 5 commits April 16, 2020 09:29
This is a problem because it makes it impossible to later
do a access_token request without a redirect_uri.
This is important because at AuthCodeGrant line 211 the redirect_uri of
the code payload is checked. Since that payload can never be empty it
makes it impossible to request a access_token without a redirect_uri.
…toryInterface::getNewToken() to help issue AccessToken with the correct permissions
@Richard87
Copy link
Author

I forced push my commit without a merge-commit I found, removed a bunch of unrelated commits to the PR, but there are still 4 commits in the PR I don't understand comes from!

@Richard87 Richard87 closed this Apr 16, 2020
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