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

OAuth2 redirect should handle existing URL parameters #25

Open
ghost opened this issue Jun 7, 2014 · 4 comments
Open

OAuth2 redirect should handle existing URL parameters #25

ghost opened this issue Jun 7, 2014 · 4 comments

Comments

@ghost
Copy link

ghost commented Jun 7, 2014

If the redirect already has a query string, e.g. https://example.com/?someparam=somevalue

and I construct the authorize URL as https://delicious.com/auth/authorize?client_id=a_client_id&redirect=https%3A%2F%2Fexample.com%2F%3Fsomeparam%3Dsomevalue, I'm redirected to https://example.com/?someparam=somevalue?code=a_code rather than the correct value of https://example.com/?someparam=somevalue&code=a_code

Bonus bug: redirect is a required parameter in the documentation, but the authorize page still loads without it.

@paoloramos
Copy link

I would like to report that I'm facing the exact same issue.

@voltagex : Have you by any chance found a solution to this?

@ghost
Copy link
Author

ghost commented Aug 30, 2014

No solution here

On 28 August 2014 00:04, paoloramos [email protected] wrote:

I would like to report that I'm facing the exact same issue.

@voltagex https://github.com/voltagex : Have you by any chance found a
solution to this?


Reply to this email directly or view it on GitHub
#25 (comment)
.

@paoloramos
Copy link

@voltagex : I actually found a workaround and posted it on SO => http://stackoverflow.com/questions/25529623/how-to-handle-delicious-api-authorize-redirect-not-handling-existing-url-get-par

I hope you find it useful.

@peoplemerge
Copy link
Contributor

Should just need to urlencode the &code=a_code

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

No branches or pull requests

2 participants