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

Replace jadams entry in KEYS #1402

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Replace jadams entry in KEYS #1402

merged 1 commit into from
Jan 13, 2025

Conversation

jadams-tresys
Copy link
Contributor

No description provided.

uid [ultimate] Joshua Adams <[email protected]>
sig 3 7B3C84C0C2319C12 2025-01-13 [self-signature]
sub rsa4096 2025-01-13 [E]
sig 7B3C84C0C2319C12 2025-01-13 [self-signature]

Copy link
Member

Choose a reason for hiding this comment

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

The digest algorithm for your new key is sha256. Although that's allowed, infra recommends we use of sha512. Can you create a new key with sha512? The KEYS file mentions the gpg configuration for that.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will do. Somehow I missed that.

Copy link
Contributor

@mbeckerle mbeckerle left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Member

@stevedlawrence stevedlawrence left a comment

Choose a reason for hiding this comment

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

+1

@jadams-tresys jadams-tresys merged commit 029b2f7 into apache:main Jan 13, 2025
11 checks passed
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.

3 participants