alopresto commented on a change in pull request #4231:
URL: https://github.com/apache/nifi/pull/4231#discussion_r416245565
##########
File path:
nifi-nar-bundles/nifi-mongodb-bundle/nifi-mongodb-processors/src/main/java/org/apache/nifi/processors/mongodb/GetMongo.java
##########
@@ -85,6 +92,22 @@
.allowableValues(YES_PP, NO_PP)
.addValidator(Validator.VALID)
.build();
+ static final PropertyDescriptor USER_NAME = new
PropertyDescriptor.Builder()
+ .name("User Name")
+ .displayName("username")
+ .description("User for mongodb authentication")
+ .required(false)
+ .addValidator(Validator.VALID)
+ .sensitive(false)
+ .build();
+
+ static final PropertyDescriptor PASSWORD = new PropertyDescriptor.Builder()
+ .name("Password")
Review comment:
The `name` value should match the formatting of the _username_ field and
a `displayName` should also be present.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
