dan-s1 commented on code in PR #8038:
URL: https://github.com/apache/nifi/pull/8038#discussion_r1439065672


##########
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/EncodeContent.java:
##########
@@ -129,19 +227,20 @@ public void onTrigger(final ProcessContext context, final 
ProcessSession session
         }
     }
 
-    private static StreamCallback getStreamCallback(final boolean encode, 
final String encoding) {
-        if (encode) {
-            if (encoding.equalsIgnoreCase(BASE64_ENCODING)) {
-                return new EncodeBase64();
-            } else if (encoding.equalsIgnoreCase(BASE32_ENCODING)) {
-                return new EncodeBase32();
+    private static StreamCallback getStreamCallback(final Boolean encode, 
final String encoding,
+        final Integer lineLength, final String lineSeparator) {
+        if (Boolean.TRUE.equals(encode)) {
+            if (encoding.equalsIgnoreCase(BASE64_ENCODING.getValue())) {

Review Comment:
   ```suggestion
       private static StreamCallback getStreamCallback(final boolean encode, 
final String encoding,
           final int lineLength, final String lineSeparator) {
           if (encode) {
               if (encoding.equalsIgnoreCase(BASE64_ENCODING.getValue())) {
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to