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

apt::key: puppetlabs-apt check now the full GPG fingerprints. #579

Merged
merged 1 commit into from Mar 20, 2015

Conversation

sbadia
Copy link
Member

@sbadia sbadia commented Mar 19, 2015

Since commit 531ef9 of puppetlabs-apt, the gpg key (apt::key) is
checked, and should use the long format.

This value is validated with a regex enforcing it to only contain valid
hexadecimal characters, be precisely 8 or 16 hexadecimal characters long
and optionally prefixed with 0x for key IDs, or 40 hexadecimal
characters long for key fingerprints.

puppetlabs/puppetlabs-apt@f588f26

This patch fixes the puppet warning below:

Warning: /Apt_key[Add key: 7BD9BF62 from Apt::Source nginx]: The id should be a full fingerprint (40 characters), see README.

Since commit 531ef9 of puppetlabs-apt, the gpg key (apt::key) is
checked, and should use the long format.

This value is validated with a regex enforcing it to only contain valid
hexadecimal characters, be precisely 8 or 16 hexadecimal characters long
and optionally prefixed with 0x for key IDs, or 40 hexadecimal
characters long for key fingerprints.

puppetlabs/puppetlabs-apt@f588f26
jfryman added a commit that referenced this pull request Mar 20, 2015
apt::key: puppetlabs-apt check now the full GPG fingerprints.
@jfryman jfryman merged commit 13b2e44 into voxpupuli:master Mar 20, 2015
@jfryman
Copy link
Contributor

jfryman commented Mar 20, 2015

Excellent. Thanks!

@errygg
Copy link

errygg commented Mar 23, 2015

I'm still getting a warning with the 16 character gpg key:

Warning: /Apt_key[Add key: ABF5BD827BD9BF62 from Apt::Source nginx]: The id should be a full fingerprint (40 characters), see README.

The fix I put in was to add the 40 character key instead of the 16 character key.

@jfryman
Copy link
Contributor

jfryman commented Mar 23, 2015

@errygg Ok. Great... misunderstood. Can you re-open and add tests then?

@sbadia
Copy link
Member Author

sbadia commented Mar 24, 2015

erf… sorry for that…

Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this pull request Mar 7, 2019
apt::key: puppetlabs-apt check now the full GPG fingerprints.
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.

None yet

3 participants