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 Java version provided to the SonarScanner 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;
 
@@ -70,6 +74,11 @@ public boolean execute(List<String> args, @Nullable String input, Consumer<Strin
       var exitCode = process.waitFor();
       stdoutConsummer.join();
       stdErrConsummer.join();
+
+      if (stdErrConsummer.getMessages().stream().anyMatch(msg -> msg.contains("UnsupportedClassVersionError"))) {
+        LOG.error(JRE_VERSION_ERROR);
+      }
+
       if (exitCode != 0) {
         LOG.debug("Java command exited with code {}", process.exitValue());
         return false;
@@ -88,16 +97,25 @@ public Path getJavaExecutable() {
   private static class StreamGobbler extends Thread {
     private final InputStream inputStream;
     private final Consumer<String> consumer;
+    private final List<String> messages;
 
     public StreamGobbler(InputStream inputStream, Consumer<String> consumer) {
       this.inputStream = inputStream;
       this.consumer = consumer;
+      this.messages = new ArrayList<>();
     }
 
     @Override
     public void run() {
       new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8)).lines()
-        .forEach(consumer);
+        .forEach(line -> {
+          messages.add(line);
+          consumer.accept(line);
+        });
+    }
+
+    private List<String> getMessages() {
+      return messages;
     }
   }
 }

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

@@ -379,15 +379,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"));
   }