Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Make hardcoded credential-osxkeychain workaround opt-out instead of opt-in #47

Merged
merged 2 commits into from
Jul 5, 2016

Conversation

olivierdagenais
Copy link
Contributor

Also expand to any version of Git that makes git-credential-osxkeychain available, not just "Apple Git". This covers the Git OSX Installer which has a more "traditional" version string, yet behaves the same with respect to hardcoding the osxkeychain credential helper first.

Manual testing

For each of the following distributions of Git:

  1. Apple
  2. Homebrew
  3. OSX Installer

...performed a git fetch against a Visual Studio Team Services Git repository and kept a close eye on the Keychain application. For Apple and OSX Installer, an entry would briefly appear in the Keychain before being deleted by the GCM4ML. In the Homebrew case, no entry appeared. In all cases, the fetch succeeded.

Mission accomplished!

Oli Dagenais added 2 commits June 30, 2016 22:38
The "Apple Git" workaround can still be disabled
by configuring it to FALSE.
It turns out the "Git OSX Installer" brings the same hardcoded
osxkeychain credential helper and thus it is no longer sufficient
to try to detect based on the Git version.
@olivierdagenais olivierdagenais merged commit 40d668f into master Jul 5, 2016
@olivierdagenais olivierdagenais deleted the osxkeychain_workaround_opt-out branch July 5, 2016 12:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants