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

Look for GPG agent socket also in $XDG_RUNTIME_DIR/gnupg #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tomaszduda23
Copy link

Signed-off-by: Tomasz Duda <[email protected]>
Signed-off-by: Tomasz Duda <[email protected]>
gpgagent.go Outdated Show resolved Hide resolved
@felixfontein felixfontein changed the title fix gpg agent connection Look for GPG agent socket also in $XDG_RUNTIME_DIR/.gnupg Jan 2, 2024
Signed-off-by: Tomasz Duda <[email protected]>
@felixfontein felixfontein requested a review from a team January 2, 2024 19:02
@tomaszduda23 tomaszduda23 changed the title Look for GPG agent socket also in $XDG_RUNTIME_DIR/.gnupg Look for GPG agent socket also in $XDG_RUNTIME_DIR/gnupg Jan 4, 2024
@tomaszduda23
Copy link
Author

@felixfontein any chance to get it merged?

@felixfontein
Copy link

@getsops/maintainers can someone else please take a look at this? I think this looks fine, but I don't know what impact //go:build !appengine has for example. Thanks.

Signed-off-by: tomaszduda23 <[email protected]>
@tomaszduda23
Copy link
Author

//go:build !appengine was removed since it is not required.

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.

2 participants