-
Notifications
You must be signed in to change notification settings - Fork 447
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 GitHub sign in with a browser #1160
base: master
Are you sure you want to change the base?
Conversation
As of September 8 2021 github removed API authentication through query parameters - https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change worked for me
I just had to dig through several issue threads and find the instructions on how to apply this fix myself. It's been over half a year and this basic functionality is still busted. Please fix this already. |
Can we get this merged and a patch release cut? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be merged, I know this repository has been deprecated but it worked perfectly
I can confirm that this bug is still present, and this fix works. Please merge. |
I can also confirm that this bug is still present and that applying this fixes it |
As of September 8 2021 github removed API authentication through query parameters - https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param/
Requirements
Description of the Change
As of September 8 2021 github removed API authentication through query parameters - https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param/ .
Changing to auth via "Authorization: token ..." header.
Alternate Designs
Benefits
Possible Drawbacks
Applicable Issues