exceptionfactory commented on code in PR #9304:
URL: https://github.com/apache/nifi/pull/9304#discussion_r1772434474


##########
nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java:
##########
@@ -380,7 +388,19 @@ synchronized void initializeServer(final ProcessContext 
context) throws Exceptio
         final String clientAuthValue = 
context.getProperty(CLIENT_AUTH).getValue();
         final Server server = createServer(context);
 
-        final StandardServerConnectorFactory serverConnectorFactory = new 
StandardServerConnectorFactory(server, port);
+        NiFiProperties nifiProperties = 
NiFiProperties.createBasicNiFiProperties(null);

Review Comment:
   Thanks @mosermw! I certainly appreciate not wanting to introduce more 
properties than necessary.
   
   I can also understand the confusion at times, particularly when it comes to 
HTTP and TLS properties. Thanks for working on the changes.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to