Github user mcgilman commented on the issue:

    https://github.com/apache/nifi/pull/1546
  
    Hey @scottyaslan,
    
    This is a great start, but I think there are a few fundamental issues with 
how `PropertyDescriptors` relay what sort of editor should be used. There does 
not appear to be a cross-browser support for masking `textareas` (multi-line 
input). There exists sensitive properties that currently expect the field to 
support multi-line input. The `input[type=password]` only supports a single 
line.
    
    I think we would need to introduce a new attribute of `PropetyDescriptor` 
which would drive whether or not to use a masked single line input or the 
existing multi-line `textarea`. The existing multi-line `textareas` support 
sensitive by not returning/showing the value once set. We would not be able to 
support a `PropertyDescriptor` that requires masking and multi-line input.
    
    Additional complications exist for sensitive properties that support 
expression language. We'd have to spend some time to determine if `CodeMirror` 
supports masking or if it's possible to introduce that sort of capability. If 
not, we would not be able to support a `PropertyDescriptor` that requires 
masking and supports expression language.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to