Enforce disableClassPathURLCheck when using default JVM args, close g…

…atling/gatling#3677

Modification:

Current workaround is only used when providing custom args.

Modification:

Always add disableClassPathURLCheck

Result:

Should work with buggy Debian stable JDK8 package

src/main/java/io/gatling/mojo/GatlingMojo.java

@@ -42,7 +42,6 @@
 import static io.gatling.mojo.MojoConstants.*;
 import static java.nio.file.StandardCopyOption.COPY_ATTRIBUTES;
 import static java.nio.file.StandardCopyOption.REPLACE_EXISTING;
-import static java.util.Arrays.asList;
 
 /**
  * Mojo to execute Gatling.
@@ -341,7 +340,7 @@ private List<String> compilerJvmArgs() {
     return computeArgs(compilerJvmArgs, COMPILER_JVM_ARGS, overrideCompilerJvmArgs);
   }
 
-  private List<String> computeArgs(List<String> custom, List<String> defaults, boolean override) {
+  private List<String> computeArgs0(List<String> custom, List<String> defaults, boolean override) {
     if (custom.isEmpty()) {
       return defaults;
     }
@@ -350,8 +349,11 @@ private List<String> computeArgs(List<String> custom, List<String> defaults, boo
       merged.addAll(defaults);
       return merged;
     }
+    return custom;
+  }
 
-    List<String> result = new ArrayList<>(custom);
+  private List<String> computeArgs(List<String> custom, List<String> defaults, boolean override) {
+    List<String> result = new ArrayList<>(computeArgs0(custom, defaults, override));
     // force disable disableClassPathURLCheck because Debian messed up and takes forever to fix, see https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=911925
     result.add("-Djdk.net.URLClassPath.disableClassPathURLCheck=true");
     return result;