From 5f65ca443764087f83fe3a2b2ac274eeea7c2699 Mon Sep 17 00:00:00 2001 From: Paul Schaub Date: Thu, 7 Apr 2022 20:28:45 +0200 Subject: [PATCH] Remove workaround for BC not properly parsing RevocationKey subpacket --- .../signature/subpackets/SignatureSubpacketsUtil.java | 9 --------- 1 file changed, 9 deletions(-) diff --git a/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketsUtil.java b/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketsUtil.java index 5a839bcb..9ebc03e8 100644 --- a/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketsUtil.java +++ b/pgpainless-core/src/main/java/org/pgpainless/signature/subpackets/SignatureSubpacketsUtil.java @@ -596,15 +596,6 @@ public final class SignatureSubpacketsUtil { } org.bouncycastle.bcpg.SignatureSubpacket last = allPackets[allPackets.length - 1]; - - if (type == SignatureSubpacket.revocationKey) { - // RevocationKey subpackets are not castable for some reason - // See https://github.com/bcgit/bc-java/pull/1085 for an upstreamed fix - // We need to manually construct the new object for now. - // TODO: Remove workaround when BC 1.71 is released (and has our fix) - return (P) new RevocationKey(last.isCritical(), last.isLongLength(), last.getData()); - } - return (P) last; }