Skip to content

Commit d47bb38

Browse files
authored
Merge pull request #266 from caarlos0/issuer-key-id
Make Issuer Key ID signature subpacket non-critical
2 parents 44ef98c + 756ebbd commit d47bb38

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

openpgp/packet/signature.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -1288,7 +1288,9 @@ func (sig *Signature) buildSubpackets(issuer PublicKey) (subpackets []outputSubp
12881288
if sig.IssuerKeyId != nil && sig.Version == 4 {
12891289
keyId := make([]byte, 8)
12901290
binary.BigEndian.PutUint64(keyId, *sig.IssuerKeyId)
1291-
subpackets = append(subpackets, outputSubpacket{true, issuerSubpacket, true, keyId})
1291+
// Note: making this critical breaks RPM <=4.16.
1292+
// See: https://github.com/ProtonMail/go-crypto/issues/263
1293+
subpackets = append(subpackets, outputSubpacket{true, issuerSubpacket, false, keyId})
12921294
}
12931295
// Notation Data
12941296
for _, notation := range sig.Notations {

0 commit comments

Comments
 (0)