SCANJLIB-246 Improve the error message when the JRE provisioning is bypassed and uses a version of Java too old

Author: antoine-vinot-sonarsource

lib/src/main/java/org/sonarsource/scanner/lib/internal/facade/forked/JavaRunner.java

@@ -36,6 +36,10 @@
 public class JavaRunner {
   private static final Logger LOG = LoggerFactory.getLogger(JavaRunner.class);
 
+  private static final String JRE_VERSION_ERROR = "The version of the custom JRE provided to the SonarScanner using the 'sonar.scanner.javaExePath' parameter is incompatible " +
+    "with your SonarQube target. You may need to upgrade the version of Java that executes the scanner. " +
+    "Refer to https://docs.sonarsource.com/sonarqube-community-build/analyzing-source-code/scanners/scanner-environment/general-requirements/ for more details.";
+
   private final Path javaExecutable;
   private final JreCacheHit jreCacheHit;
 
@@ -68,6 +72,11 @@ public boolean execute(List<String> args, @Nullable String input, Consumer<Strin
       var exitCode = process.waitFor();
       stdoutConsummer.join();
       stdErrConsummer.join();
+
+      if (stdErrConsummer.hasUnsupportedClassVersionError()) {
+        LOG.error(JRE_VERSION_ERROR);
+      }
+
       if (exitCode != 0) {
         LOG.debug("Java command exited with code {}", process.exitValue());
         return false;
@@ -86,6 +95,7 @@ public Path getJavaExecutable() {
   private static class StreamGobbler extends Thread {
     private final InputStream inputStream;
     private final Consumer<String> consumer;
+    private boolean unsupportedClassVersionError = false;
 
     public StreamGobbler(InputStream inputStream, Consumer<String> consumer) {
       this.inputStream = inputStream;
@@ -95,7 +105,17 @@ public StreamGobbler(InputStream inputStream, Consumer<String> consumer) {
     @Override
     public void run() {
       new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8)).lines()
-        .forEach(consumer);
+        .forEach(line -> {
+          if (line.contains("UnsupportedClassVersionError")) {
+            unsupportedClassVersionError = true;
+          }
+          consumer.accept(line);
+        });
     }
+
+    public boolean hasUnsupportedClassVersionError() {
+      return unsupportedClassVersionError;
+    }
+
   }
 }

lib/src/test/java/org/sonarsource/scanner/lib/ScannerEngineBootstrapperTest.java

@@ -403,15 +403,15 @@ void should_set_deprecated_ssl_properties() {
 
     when(httpConfig.getSslConfig())
       .thenReturn(new SslConfig(
-        new CertificateStore(Paths.get("some/keystore.p12"), "keystorePass"),
-        new CertificateStore(Paths.get("some/truststore.p12"), "truststorePass")));
+        new CertificateStore(Paths.get("some", "keystore.p12"), "keystorePass"),
+        new CertificateStore(Paths.get("some", "truststore.p12"), "truststorePass")));
 
     underTest.adaptDeprecatedPropertiesForInProcessBootstrapping(Map.of(), httpConfig);
 
     assertThat(System.getProperties()).contains(
-      entry("javax.net.ssl.keyStore", "some/keystore.p12"),
+      entry("javax.net.ssl.keyStore", Paths.get("some", "keystore.p12").toString()),
       entry("javax.net.ssl.keyStorePassword", "keystorePass"),
-      entry("javax.net.ssl.trustStore", "some/truststore.p12"),
+      entry("javax.net.ssl.trustStore", Paths.get("some", "truststore.p12").toString()),
       entry("javax.net.ssl.trustStorePassword", "truststorePass"));
   }