Support pipe and new line in the 'No proxy host'

README.md

@@ -188,7 +188,7 @@ If you're using a JFrog platform that's situated behind an HTTP/S proxy, you sho
 under `Manage Jenkins` > `Manage Plugins` > `Advanced`.
 
 To exclude the JFrog platform from going through a configured proxy, provide your JFrog platform's host details in
-the `No Proxy Host` section. This should be a list of comma-separated hosts.
+the `No Proxy Host` section.
 Notice that the JFrog CLI is typically downloaded from releases.jfrog.io. You may need to add that to your list as well.
 
 ## Jenkins Configuration as Code

src/main/java/io/jenkins/plugins/jfrog/CliEnvConfigurator.java

@@ -5,7 +5,6 @@
 import io.jenkins.plugins.jfrog.configuration.JenkinsProxyConfiguration;
 import org.apache.commons.lang3.StringUtils;
 
-
 /**
  * Configures JFrog CLI environment variables for the job.
  *

src/test/java/io/jenkins/plugins/jfrog/CliEnvConfiguratorNoProxyTest.java → src/test/java/io/jenkins/plugins/jfrog/NoProxyExtractorTest.java

@@ -14,19 +14,18 @@
  * @author nathana
  **/
 @RunWith(Parameterized.class)
-public class CliEnvConfiguratorNoProxyTest {
+public class NoProxyExtractorTest {
     private final String noProxyList;
     private final String expectedResult;
 
-    public CliEnvConfiguratorNoProxyTest(String noProxyList, String expectedResult) {
+    public NoProxyExtractorTest(String noProxyList, String expectedResult) {
         this.noProxyList = noProxyList;
         this.expectedResult = expectedResult;
     }
 
     @Parameterized.Parameters
     public static Collection<Object[]> dataProvider() {
         return Arrays.asList(
-                // Positive tests
                 new Object[]{"artifactory.jfrog.io", "artifactory.jfrog.io"},
                 new Object[]{"artifactory.jfrog.io    \n      artifactory1.jfrog.io          ", "artifactory.jfrog.io;artifactory1.jfrog.io"},
                 new Object[]{"   artifactory.jfrog.io    \n  \r     artifactory1.jfrog.io;artifactory2.jfrog.io    \n      artifactory3.jfrog.io | artifactory4.jfrog.io    \n      artifactory5.jfrog.io ", "artifactory.jfrog.io;artifactory1.jfrog.io;artifactory2.jfrog.io;artifactory3.jfrog.io;artifactory4.jfrog.io;artifactory5.jfrog.io"},
@@ -41,7 +40,7 @@ public static Collection<Object[]> dataProvider() {
     }
 
     @Test
-    public void testValidateCliVersion() {
+    public void testNoProxyExtractor() {
         assertEquals(expectedResult, noProxyExtractor(noProxyList));
     }
 }