exceptionfactory commented on code in PR #6057: URL: https://github.com/apache/nifi/pull/6057#discussion_r879674931
########## nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessorResource.java: ########## @@ -423,6 +429,11 @@ public Response getPropertyDescriptor( // get the property descriptor final PropertyDescriptorDTO descriptor = serviceFacade.getProcessorPropertyDescriptor(id, propertyName); + // Adjust sensitive status for dynamic properties based on requested status + if (descriptor.isDynamic()) { + descriptor.setSensitive(sensitive); + } Review Comment: Thanks for catching that @markap14, that's a good point. Since there are a couple components that mark dynamic properties as sensitive based on the property name, this approach would override that setting. Will make the adjustment as suggested. -- 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