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

support for age identity with passphrase #1400

Merged
merged 7 commits into from
Feb 24, 2025
Merged

Conversation

tomaszduda23
Copy link
Contributor

@tomaszduda23 tomaszduda23 commented Dec 30, 2023

Solves #933. It is basically rip of from https://github.com/FiloSottile/age/blob/101cc8676386b0503571a929a88618cae2f0b1cd/cmd/age/parse.go#L143. It looks that sops already support reading passwords similar way for PGP

It also support password prompt by gpg-agent. It requires merge of getsops/gopgagent#1

@tomaszduda23 tomaszduda23 changed the title add support for passphrase for age support for age identity with passphrase Dec 31, 2023
Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Converting to draft until the agent from getsops is used.

…-dfc8ceccf9

build(deps): Bump the go group with 7 updates
@tomaszduda23
Copy link
Contributor Author

@felixfontein would you mind review again?

Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Mering #1692 introduced conflicts in keysource_test.go; my guess is that both PRs added something at the end which caused this, so it should be easy to fix.

Signed-off-by: Tomasz Duda <tomaszduda23@gmail.com>
@tomaszduda23
Copy link
Contributor Author

Mering #1692 introduced conflicts in keysource_test.go; my guess is that both PRs added something at the end which caused this, so it should be easy to fix.

Fixed.

Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for updating the PR!

Signed-off-by: Tomasz Duda <tomaszduda23@gmail.com>
Signed-off-by: Tomasz Duda <tomaszduda23@gmail.com>
@felixfontein
Copy link
Contributor

(I'll wait until tomorrow with merging this to give others a chance to comment. In case anyone else wants to take a look, and doesn't have time til then, please write it here, then I'll wait longer. CC @getsops/maintainers.)

@felixfontein felixfontein merged commit beb0d87 into getsops:main Feb 24, 2025
15 checks passed
@felixfontein
Copy link
Contributor

@tomaszduda23 thanks a lot for your contribution and patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants