Skip to content

Commit 756ebbd

Browse files
caarlos0twiss
authored andcommitted
Make Issuer Key ID signature subpacket non-critical
RPM <=4.16 does not support it.
1 parent 44ef98c commit 756ebbd

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)