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

Remove username/password authentication for GitHub.com #171

Merged
merged 2 commits into from
Sep 21, 2020

Conversation

mjcheetham
Copy link
Collaborator

Now that we have the GCM OAuth application in the allow-list for GitHub apps, we no longer need to offer username/password authentication for GitHub.com!

Now that we have the GCM OAuth application in the allow-list for GitHub
apps, we no longer need to offer username/password authentication for
GitHub.com!
@mjcheetham mjcheetham added the host:github Specific to the GitHub host provider label Sep 18, 2020
Fix a typo in a constant variable indicating the available
authentication modes for GitHub.COM.
@mjcheetham mjcheetham merged commit 368d65a into git-ecosystem:master Sep 21, 2020
@mjcheetham mjcheetham deleted the gh-oauthonly branch September 21, 2020 10:24
@AlexanderLanin
Copy link
Contributor

Sorry, I'm new here. Shouldn't this change also remove this?
https://github.com/microsoft/Git-Credential-Manager-Core/blob/c3a543a1352dba953e027f7a92ee4a94b2293a22/src/shared/GitHub/GitHubHostProvider.cs#L98

And therefore this?
https://github.com/microsoft/Git-Credential-Manager-Core/blob/c3a543a1352dba953e027f7a92ee4a94b2293a22/src/shared/GitHub/GitHubHostProvider.cs#L129

Maybe it can be reused for GitHub Enterprise with ADFS somehow. But so far I don't see how. That would be quite nice since for me GCM is not (yet) allowed via OAuth. And now password login is gone for creating PATs as well.

@mjcheetham
Copy link
Collaborator Author

Hi @AlexanderLanin,

But so far I don't see how. That would be quite nice since for me GCM is not (yet) allowed via OAuth. And now password login is gone for creating PATs as well.

You should be able to set the GCM_GITHUB_AUTHMODES environment variable or the Git config setting credential.gitHubAuthModes to "basic oauth" to enable both options again.

Also, since username/password support has been removed from the GitHub server, you'd need to manually generate a PAT and enter that as the password in the prompt.

Let me know if there are any issues with this workaround and create a new issue. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
host:github Specific to the GitHub host provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants