exceptionfactory commented on a change in pull request #5574:
URL: https://github.com/apache/nifi/pull/5574#discussion_r763564415
##########
File path:
nifi-nar-bundles/nifi-websocket-bundle/nifi-websocket-services-jetty/src/main/java/org/apache/nifi/websocket/jetty/JettyWebSocketClient.java
##########
@@ -145,6 +145,19 @@
.defaultValue("US-ASCII")
.build();
+ public static final PropertyDescriptor CUSTOM_AUTH = new
PropertyDescriptor.Builder()
+ .name("custom-authorization")
+ .displayName("Custom Authorization")
+ .description(
+ "If set tgether with \"User Name\" and \"User Password\",
instead of using Basic" +
+ " Authentication the value of the property will be
assigned to the \"Authorization\" HTTP header.")
Review comment:
Instead of describing the behavior this way, what do you think about
extending the `customValidate()` method to ensure that setting this property
excludes setting `User Name` and `User Password`, and vice versa? Checking that
both this property and the other credentials properties are not set would help
avoid potential confusion in the component configuration.
--
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]
