Github user alopresto commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2684#discussion_r186503174
--- Diff:
nifi-toolkit/nifi-toolkit-tls/src/main/java/org/apache/nifi/toolkit/tls/standalone/TlsToolkitStandalone.java
---
@@ -236,6 +236,6 @@ public void
createNifiKeystoresAndTrustStores(StandaloneConfig standaloneConfig)
}
protected static String getClientDnFile(String clientDn) {
- return clientDn.replace(',', '_').replace(' ', '_');
+ return clientDn.replace(',', '_').replace(' ',
'_').replaceAll("[^\\w&&[^\\.]&&[^=]]", "_");
--- End diff --
Since the regular expression is being used in multiple places, I would
recommend making this a `public static final` constant in one of the toolkit
utility classes.
---
