Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix multiple gitHubAuthModes #278

Merged
merged 1 commit into from
Feb 3, 2021

Conversation

AlexanderLanin
Copy link
Contributor

List must be comma separated.

Before:

$ GCM_TRACE=1 git fetch
22:22:08.037216 ...\Application.cs:69   trace: [RunInternalAsync] Git Credential Manager version 2.0.318-beta+44acfafa98 (Windows, .NET Framework 4.0.30319.42000) 'get'
22:22:08.040208 ...\Command.cs:63       trace: [ExecuteAsync] Start 'get' command...
22:22:08.046193 ...\Command.cs:74       trace: [ExecuteAsync] Detecting host provider for input:
22:22:08.047214 ...\Command.cs:75       trace: [ExecuteAsync]   protocol=https
22:22:08.047214 ...\Command.cs:75       trace: [ExecuteAsync]   host=***
22:22:08.206763 ...viderRegistry.cs:129 trace: [GetProvider] Performing auto-detection of host provider.
22:22:08.208757 ...\Command.cs:77       trace: [ExecuteAsync] Host provider 'GitHub' was selected.
22:22:08.211749 ...\HostProvider.cs:115 trace: [GetCredentialAsync] Looking for existing credential in store with service=https://*** account=...
22:22:08.214770 ...\HostProvider.cs:120 trace: [GetCredentialAsync] No existing credentials found.
22:22:08.214770 ...\HostProvider.cs:123 trace: [GetCredentialAsync] Creating new credential...
22:22:08.266604 ...bHostProvider.cs:227 trace: [GetSupportedAuthenticationModesAsync] Invalid value for supported authentication modes override setting: 'OAuth Basic'

After:

$ GCM_TRACE=1 git fetch
22:22:32.792872 ...\Application.cs:69   trace: [RunInternalAsync] Git Credential Manager version 2.0.318-beta+44acfafa98 (Windows, .NET Framework 4.0.30319.42000) 'get'
22:22:32.796891 ...\Command.cs:63       trace: [ExecuteAsync] Start 'get' command...
22:22:32.801849 ...\Command.cs:74       trace: [ExecuteAsync] Detecting host provider for input:
22:22:32.802847 ...\Command.cs:75       trace: [ExecuteAsync]   protocol=https
22:22:32.802847 ...\Command.cs:75       trace: [ExecuteAsync]   host=***
22:22:32.945493 ...viderRegistry.cs:129 trace: [GetProvider] Performing auto-detection of host provider.
22:22:32.947488 ...\Command.cs:77       trace: [ExecuteAsync] Host provider 'GitHub' was selected.
22:22:32.949455 ...\HostProvider.cs:115 trace: [GetCredentialAsync] Looking for existing credential in store with service=https://*** account=...
22:22:32.951449 ...\HostProvider.cs:120 trace: [GetCredentialAsync] No existing credentials found.
22:22:32.951449 ...\HostProvider.cs:123 trace: [GetCredentialAsync] Creating new credential...
22:22:32.991343 ...bHostProvider.cs:222 trace: [GetSupportedAuthenticationModesAsync] Supported authentication modes override present: Basic, OAuth

List must be comma separated.

Before:
```
$ GCM_TRACE=1 git fetch
22:22:08.037216 ...\Application.cs:69   trace: [RunInternalAsync] Git Credential Manager version 2.0.318-beta+44acfafa98 (Windows, .NET Framework 4.0.30319.42000) 'get'
22:22:08.040208 ...\Command.cs:63       trace: [ExecuteAsync] Start 'get' command...
22:22:08.046193 ...\Command.cs:74       trace: [ExecuteAsync] Detecting host provider for input:
22:22:08.047214 ...\Command.cs:75       trace: [ExecuteAsync]   protocol=https
22:22:08.047214 ...\Command.cs:75       trace: [ExecuteAsync]   host=***
22:22:08.206763 ...viderRegistry.cs:129 trace: [GetProvider] Performing auto-detection of host provider.
22:22:08.208757 ...\Command.cs:77       trace: [ExecuteAsync] Host provider 'GitHub' was selected.
22:22:08.211749 ...\HostProvider.cs:115 trace: [GetCredentialAsync] Looking for existing credential in store with service=https://*** account=...
22:22:08.214770 ...\HostProvider.cs:120 trace: [GetCredentialAsync] No existing credentials found.
22:22:08.214770 ...\HostProvider.cs:123 trace: [GetCredentialAsync] Creating new credential...
22:22:08.266604 ...bHostProvider.cs:227 trace: [GetSupportedAuthenticationModesAsync] Invalid value for supported authentication modes override setting: 'OAuth Basic'
```

After:
```
$ GCM_TRACE=1 git fetch
22:22:32.792872 ...\Application.cs:69   trace: [RunInternalAsync] Git Credential Manager version 2.0.318-beta+44acfafa98 (Windows, .NET Framework 4.0.30319.42000) 'get'
22:22:32.796891 ...\Command.cs:63       trace: [ExecuteAsync] Start 'get' command...
22:22:32.801849 ...\Command.cs:74       trace: [ExecuteAsync] Detecting host provider for input:
22:22:32.802847 ...\Command.cs:75       trace: [ExecuteAsync]   protocol=https
22:22:32.802847 ...\Command.cs:75       trace: [ExecuteAsync]   host=***
22:22:32.945493 ...viderRegistry.cs:129 trace: [GetProvider] Performing auto-detection of host provider.
22:22:32.947488 ...\Command.cs:77       trace: [ExecuteAsync] Host provider 'GitHub' was selected.
22:22:32.949455 ...\HostProvider.cs:115 trace: [GetCredentialAsync] Looking for existing credential in store with service=https://*** account=...
22:22:32.951449 ...\HostProvider.cs:120 trace: [GetCredentialAsync] No existing credentials found.
22:22:32.951449 ...\HostProvider.cs:123 trace: [GetCredentialAsync] Creating new credential...
22:22:32.991343 ...bHostProvider.cs:222 trace: [GetSupportedAuthenticationModesAsync] Supported authentication modes override present: Basic, OAuth
```
Copy link
Collaborator

@mjcheetham mjcheetham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes! Good spot here, thanks 👍

@mjcheetham mjcheetham merged commit 27c0470 into git-ecosystem:master Feb 3, 2021
@mjcheetham mjcheetham mentioned this pull request Mar 3, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants