[GH-ISSUE #145] Bug when extracting the parameters from the query string #90

Closed
opened 2026-03-03 16:45:35 +03:00 by kerem · 2 comments
Owner

Originally created by @mohwaili on GitHub (Nov 10, 2015).
Original GitHub issue: https://github.com/OAuthSwift/OAuthSwift/issues/145

Ampersand in the parameters dictionary after authenticating. Off course the authentication will always fail.

bug

Originally created by @mohwaili on GitHub (Nov 10, 2015). Original GitHub issue: https://github.com/OAuthSwift/OAuthSwift/issues/145 Ampersand in the parameters dictionary after authenticating. Off course the authentication will always fail. ![bug](https://cloud.githubusercontent.com/assets/9988884/11068121/33a00f92-87d0-11e5-8516-d0958ac08805.png)
kerem 2026-03-03 16:45:35 +03:00
  • closed this issue
  • added the
    wontfix
    label
Author
Owner

@phimage commented on GitHub (Nov 10, 2015):

I think callback url is malformed : ..avans?&oauth... must be ..avans?oauth...
Maybe url RFC allow it, but I think it's bad
So I think nothing will be done into OAuthSwift code to support that

Maybe the workaround is change your callback url by adding a parameter
..avans?dummy=1
and maybe the website will return a callback url like that ..avans?dummy=1&oauth...

<!-- gh-comment-id:155527474 --> @phimage commented on GitHub (Nov 10, 2015): I think callback url is malformed : ..avans?&oauth... must be ..avans?oauth... Maybe url RFC allow it, but I think it's bad So I think nothing will be done into OAuthSwift code to support that Maybe the workaround is change your callback url by adding a parameter ..avans?dummy=1 and maybe the website will return a callback url like that ..avans?dummy=1&oauth...
Author
Owner

@mohwaili commented on GitHub (Nov 10, 2015):

@phimage thank you, adding the dummy=1 fixed the problem

<!-- gh-comment-id:155582236 --> @mohwaili commented on GitHub (Nov 10, 2015): @phimage thank you, adding the dummy=1 fixed the problem
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/OAuthSwift#90
No description provided.