bugfix: Make sure proxy settings are being forwarded to coursier

scalameta · Sep 13, 2024 · 53be2c2 · 53be2c2
1 parent bc75810
commit 53be2c2
Show file tree

Hide file tree

Showing 4 changed files with 48 additions and 13 deletions.
diff --git a/packages/metals-languageclient/src/fetchMetals.ts b/packages/metals-languageclient/src/fetchMetals.ts
@@ -2,6 +2,7 @@ import * as semver from "semver";
 import { ChildProcessPromise, spawn } from "promisify-child-process";
 import { JavaConfig } from "./getJavaConfig";
 import { OutputChannel } from "./interfaces/OutputChannel";
+import { convertToCoursierProperties } from "./getJavaOptions";
 
 interface FetchMetalsOptions {
   serverVersion: string;
@@ -70,15 +71,10 @@ export async function fetchMetals({
     ].concat(coursierArgs);
     return { promise: spawn(javaPath, jarArgs, environment) };
   } else {
-    // Convert Java properties to the "-J" argument form used by Coursier
-    var javaArgs: Array<string> = [];
-
-    // setting properties on windows native launcher doesn't work
-    if (process.platform != "win32")
-      javaArgs = javaOptions
-        .concat(["-Dfile.encoding=UTF-8"])
-        .concat(fetchProperties)
-        .map((p) => `-J${p}`);
+    const javaArgs: Array<string> = convertToCoursierProperties(
+      javaOptions.concat(["-Dfile.encoding=UTF-8"]).concat(fetchProperties),
+      false
+    );
 
     return {
       promise: spawn(coursier, javaArgs.concat(coursierArgs), environment),

diff --git a/packages/metals-languageclient/src/getJavaOptions.ts b/packages/metals-languageclient/src/getJavaOptions.ts
@@ -19,6 +19,26 @@ function sanitizeOption(option: string): string {
   return option.trim();
 }
 
+export function convertToCoursierProperties(
+  serverProperties: string[],
+  isCoursierJar: boolean
+): string[] {
+  // setting properties on windows native launcher doesn't work
+  if (!isCoursierJar && process.platform == "win32") {
+    return [];
+  } else {
+    return serverProperties
+      .filter((prop) => isValidOption(prop))
+      .map((prop) => {
+        if (isCoursierJar) {
+          return prop;
+        } else {
+          return "-J" + prop;
+        }
+      });
+  }
+}
+
 function isValidOption(option: string): boolean {
   return (
     option.startsWith("-") &&
@@ -28,6 +48,7 @@ function isValidOption(option: string): boolean {
     !option.startsWith("-Xms") &&
     !option.startsWith("-Xmx") &&
     !option.startsWith("-Xss") &&
+    !option.startsWith("-XX:") &&
     // Do not alter stdout that we capture when using Coursier
     option !== "-XX:+PrintCommandLineFlags"
   );

diff --git a/packages/metals-languageclient/src/setupCoursier.ts b/packages/metals-languageclient/src/setupCoursier.ts
@@ -9,6 +9,7 @@ import { OutputChannel } from "./interfaces/OutputChannel";
 import path from "path";
 import fs from "fs";
 import { findOnPath } from "./util";
+import { convertToCoursierProperties } from "./getJavaOptions";
 
 const coursierVersion = "v2.1.8";
 // https://github.com/coursier/launchers contains only launchers with the most recent version
@@ -20,7 +21,8 @@ export async function setupCoursier(
   coursierFetchPath: string,
   extensionPath: string,
   output: OutputChannel,
-  forceCoursierJar: boolean
+  forceCoursierJar: boolean,
+  serverProperties: string[]
 ): Promise<{ coursier: string; javaHome: string }> {
   const handleOutput = (out: Buffer) => {
     const msg = "\t" + out.toString().trim().split("\n").join("\n\t");
@@ -54,15 +56,30 @@ export async function setupCoursier(
   };
 
   const resolveJavaHomeWithCoursier = async (coursier: string) => {
+    const nonJvmServerProperties = convertToCoursierProperties(
+      serverProperties,
+      coursier.endsWith(".jar")
+    );
     await run(
       coursier,
-      ["java", "--jvm", `temurin:${javaVersion}`, "-version"],
+      [
+        "java",
+        ...nonJvmServerProperties,
+        "--jvm",
+        `temurin:${javaVersion}`,
+        "-version",
+      ],
       handleOutput
     );
 
     const getJavaPath = spawn(
       coursier,
-      ["java-home", "--jvm", `temurin:${javaVersion}`],
+      [
+        "java-home",
+        ...nonJvmServerProperties,
+        "--jvm",
+        `temurin:${javaVersion}`,
+      ],
       {
         encoding: "utf8",
       }

diff --git a/packages/metals-vscode/src/extension.ts b/packages/metals-vscode/src/extension.ts
@@ -230,7 +230,8 @@ async function fetchAndLaunchMetals(
     metalsDirPath,
     context.extensionPath,
     outputChannel,
-    forceCoursierJar
+    forceCoursierJar,
+    serverProperties
   );
 
   const canRetryWithJar =