-
-
Notifications
You must be signed in to change notification settings - Fork 501
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
gopass-git-credentials: rename to git-credential-gopass #1670
Conversation
bb20071
to
cf98e2d
Compare
With this rename, we can use "gopass" as a git-credential, no need to have the hackery of invoke another shell. Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>
cf98e2d
to
73f14ee
Compare
Codecov Report
@@ Coverage Diff @@
## master #1670 +/- ##
=======================================
Coverage 57.64% 57.64%
=======================================
Files 204 204
Lines 9854 9854
=======================================
Hits 5680 5680
Misses 3503 3503
Partials 671 671
Continue to review full report at Codecov.
|
@@ -243,7 +243,7 @@ func (s *gc) Configure(c *cli.Context) error { | |||
if flags == 0 { | |||
log.Println("No target given, assuming --global.") | |||
} | |||
cmd := exec.CommandContext(ctx, "git", "config", flag, "credential.helper", `"!gopass-git-credentials $@"`) | |||
cmd := exec.CommandContext(ctx, "git", "config", flag, "credential.helper", "gopass") |
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.
Doesn't this configure the gopass
binary itself as the git credential helper?
On 2020-12-31 12:54:46-0800, Dominik Schulz ***@***.***> wrote:
@dominikschulz commented on this pull request.
> @@ -243,7 +243,7 @@ func (s *gc) Configure(c *cli.Context) error {
if flags == 0 {
log.Println("No target given, assuming --global.")
}
- cmd := exec.CommandContext(ctx, "git", "config", flag, "credential.helper", `"!gopass-git-credentials $@"`)
+ cmd := exec.CommandContext(ctx, "git", "config", flag, "credential.helper", "gopass")
Doesn't this configure the `gopass` binary itself as the git credential helper?
This configure "git-credential-gopass" as the git credential helper.
https://git-scm.com/docs/gitcredentials#Documentation/gitcredentials.txt-helper
…--
Danh
|
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.
LGTM
Thank you. Please rebase and add |
This commit removes the binaries that were migrated to their dedicated git repos. Fixes gopasspw#1673 Fixes gopasspw#1649 Fixes gopasspw#1652 Fixes gopasspw#1631 Fixes gopasspw#1165 Fixes gopasspw#1711 Fixes gopasspw#1670 Fixes gopasspw#1639 Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
This commit removes the binaries that were migrated to their dedicated git repos. Fixes gopasspw#1673 Fixes gopasspw#1649 Fixes gopasspw#1652 Fixes gopasspw#1631 Fixes gopasspw#1165 Fixes gopasspw#1711 Fixes gopasspw#1670 Fixes gopasspw#1639 RELEASE_NOTES=[CLEANUP] Remove migrated binaries Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
This commit removes the binaries that were migrated to their dedicated git repos. Fixes gopasspw#1673 Fixes gopasspw#1649 Fixes gopasspw#1652 Fixes gopasspw#1631 Fixes gopasspw#1165 Fixes gopasspw#1711 Fixes gopasspw#1670 Fixes gopasspw#1639 RELEASE_NOTES=[CLEANUP] Remove migrated binaries Signed-off-by: Dominik Schulz <dominik.schulz@gauner.org>
With this rename, we can use "gopass" as a git-credential,
no need to have the hackery of invoke another shell.