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

GitHub access token as basic auth. #431

Closed
wants to merge 1 commit into from
Closed

GitHub access token as basic auth. #431

wants to merge 1 commit into from

Conversation

PPingot
Copy link

@PPingot PPingot commented Feb 6, 2020

GitHub access token as parameter is deprecated,
basic auth is recommended instead.

GitHub access token as parameter is deprecated,
basic auth is recommended instead.
@timabbott
Copy link

Can merging this be prioritized? Today, GitHub started sending emails to any software using the old API every 3 days, which is pretty annoying and also suggests they may drop support for the interface imminently. Thanks!

@PPingot
Copy link
Author

PPingot commented Feb 6, 2020

There is a better PR for this #428. I noticed it after sending this one. #428 is simpler, does not require base64 at all.

@PPingot PPingot closed this Feb 7, 2020
@PPingot PPingot deleted the github_basic_auth branch February 7, 2020 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants