Added a builder method to sign/encrypt as a text document rather than binary data.

Author: bjansen

src/main/java/name/neuhalfen/projects/crypto/bouncycastle/openpgp/BuildEncryptionOutputStreamAPI.java

@@ -51,6 +51,7 @@ public final class BuildEncryptionOutputStreamAPI {
   private String signWith;
   private Set<PGPPublicKey> recipients;
   private boolean armorOutput;
+  private boolean textMode;
 
   // Signature
 
@@ -95,6 +96,8 @@ private KeySelectionStrategy getKeySelectionStrategy() {
 
   public interface Build {
 
+    Build textMode();
+
     OutputStream andWriteTo(OutputStream sinkForEncryptedData)
         throws PGPException, SignatureException, NoSuchAlgorithmException, NoSuchProviderException, IOException;
   }
@@ -502,6 +505,12 @@ public Build armorAsciiOutput() {
 
           public final class Builder implements Build {
 
+            @Override
+            public Build textMode() {
+              BuildEncryptionOutputStreamAPI.this.textMode = true;
+              return this;
+            }
+
             @Override
             public OutputStream andWriteTo(OutputStream sinkForEncryptedData)
                 throws PGPException, SignatureException, NoSuchAlgorithmException, NoSuchProviderException, IOException {
@@ -513,7 +522,8 @@ public OutputStream andWriteTo(OutputStream sinkForEncryptedData)
                   BuildEncryptionOutputStreamAPI.this.sinkForEncryptedData,
                   getKeySelectionStrategy(),
                   BuildEncryptionOutputStreamAPI.this.armorOutput,
-                  BuildEncryptionOutputStreamAPI.this.recipients);
+                  BuildEncryptionOutputStreamAPI.this.recipients,
+                  BuildEncryptionOutputStreamAPI.this.textMode);
 
             }
           }

src/main/java/name/neuhalfen/projects/crypto/bouncycastle/openpgp/encrypting/PGPEncryptingStream.java

@@ -78,6 +78,7 @@ private PGPEncryptingStream(final KeyringConfig config, final PGPAlgorithmSuite
    * @param keySelectionStrategy selection strategy
    * @param armor armor the file (true) or use binary.
    * @param encryptTo encrypt to
+   * @param textMode simulates GnuPG's {@code --textmode} flag
    *
    * @return stream where plaintext gets written into
    *
@@ -93,7 +94,8 @@ public static OutputStream create(final KeyringConfig config,
       final OutputStream cipherTextSink,
       final KeySelectionStrategy keySelectionStrategy,
       final boolean armor,
-      final Set<PGPPublicKey> encryptTo)
+      final Set<PGPPublicKey> encryptTo,
+      final boolean textMode)
       throws IOException, PGPException, NoSuchAlgorithmException, NoSuchProviderException {
 
     requireNonNull(config, "callback must not be null");
@@ -109,7 +111,7 @@ public static OutputStream create(final KeyringConfig config,
     }
 
     final PGPEncryptingStream encryptingStream = new PGPEncryptingStream(config, algorithmSuite);
-    encryptingStream.setup(cipherTextSink, signingUid, encryptTo, keySelectionStrategy, armor);
+    encryptingStream.setup(cipherTextSink, signingUid, encryptTo, keySelectionStrategy, armor, textMode);
     return encryptingStream;
   }
 
@@ -122,6 +124,7 @@ public static OutputStream create(final KeyringConfig config,
    * @param pubEncKeys the pub enc keys
    * @param keySelectionStrategy key selection strategy (for signatures)
    * @param armor if OutputStream should be "armored", that means base64 encoded
+   * @param textMode simulates GnuPG's {@code --textmode} flag
    *
    * @throws IOException Signals that an I/O exception has occurred.
    * @throws PGPException the pGP exception
@@ -134,7 +137,8 @@ private void setup(final OutputStream cipherTextSink,
       @Nullable final String signingUid,
       final Set<PGPPublicKey> pubEncKeys,
       final KeySelectionStrategy keySelectionStrategy,
-      final boolean armor) throws
+      final boolean armor,
+      final boolean textMode) throws
       IOException, PGPException {
     isDoSign = signingUid != null;
 
@@ -186,7 +190,7 @@ private void setup(final OutputStream cipherTextSink,
           new BcPGPContentSignerBuilder(pgpSec.getPublicKey().getAlgorithm(),
               algorithmSuite.getHashAlgorithmCode().getAlgorithmId()));
 
-      signatureGenerator.init(PGPSignature.BINARY_DOCUMENT, pgpPrivKey);
+      signatureGenerator.init(textMode ? PGPSignature.CANONICAL_TEXT_DOCUMENT : PGPSignature.BINARY_DOCUMENT, pgpPrivKey);
 
       final Iterator<?> userIDs = pgpSec.getPublicKey().getUserIDs();
       if (userIDs.hasNext()) {
@@ -208,7 +212,7 @@ private void setup(final OutputStream cipherTextSink,
 
     encryptionDataStreamGenerator = new PGPLiteralDataGenerator();
     encryptionDataStream = encryptionDataStreamGenerator
-        .open(compressionStream, PGPLiteralData.BINARY, "", new Date(), new byte[1 << 16]);
+        .open(compressionStream, textMode ? PGPLiteralData.TEXT : PGPLiteralData.BINARY, "", new Date(), new byte[1 << 16]);
   }
 
   @Override
@@ -268,4 +272,4 @@ public void close() throws IOException {
       isClosed = true;
     }
   }
-}
+}

src/test/java/name/neuhalfen/projects/crypto/bouncycastle/openpgp/encrypting/EncryptWithOpenPGPTestDriver.java

@@ -91,10 +91,10 @@ void encryptAndSign(final InputStream in, OutputStream out, final Set<PGPPublicK
 
     try (final OutputStream encryptionStream = PGPEncryptingStream
         .create(config, algorithmSuite, signatureUid, out, keySelectionStrategy, armor,
-            pubEncKeys)) {
+            pubEncKeys, false)) {
       Streams.pipeAll(in, encryptionStream);
       encryptionStream.flush();
     }
     out.flush();
   }
-}
+}