Skip to content

Allow the use of git's useHttpPath setting#131

Draft
fcwoknhenuxdfiyv wants to merge 2 commits intogopasspw:masterfrom
fcwoknhenuxdfiyv:master
Draft

Allow the use of git's useHttpPath setting#131
fcwoknhenuxdfiyv wants to merge 2 commits intogopasspw:masterfrom
fcwoknhenuxdfiyv:master

Conversation

@fcwoknhenuxdfiyv
Copy link
Copy Markdown

This PR modifies composePath to take into account the credential.Path field being populated due to useHttpPath = true in a credential stanze in the git config:

[credential "github.com"]
  useHttpPath = true

It will add the first part of the path making the assumtion that it's a user or organisation:

$ gopass list -f git
git/git.somewhere.com/auser
git/github.com/fcwoknhenuxdfiyv/fcwoknhenuxdfiyv
git/github.com/another_github_org/another_gh_user

Jason Quigley and others added 2 commits October 16, 2025 15:57
Signed-off-by: Jason Quigley <sinker.octagon0t@icloud.com>
Signed-off-by: Jason Quigley <sinker.octagon0t@icloud.com>
@dominikschulz dominikschulz self-requested a review October 17, 2025 11:42
@fcwoknhenuxdfiyv
Copy link
Copy Markdown
Author

@dominikschulz I just noticed that the tests are failing for Windows. I imagine it has something to do with the path separator. I have no Windows machines to test on.

@fcwoknhenuxdfiyv fcwoknhenuxdfiyv marked this pull request as draft October 26, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant