fix: revert base URL change (#152)

Revert "feat: add base URL preference for CLI downloads and only download CLI [HEAD-779] (#151)"

This reverts commit e0b255d.

Author: bastiandoetsch

.github/workflows/ci.yml

@@ -37,14 +37,14 @@ jobs:
 
     - name: Build with Maven
       if: ${{ matrix.os == 'ubuntu-latest' }}
-      run: ./mvnw clean verify -P sign -DtrimStackTrace=false
+      run: ./mvnw clean verify -P sign
       env:
         KEYSTORE_PASS: ${{ secrets.KEYSTORE_PASS }}
 
     - name: Build with Maven
       if: ${{ matrix.os != 'ubuntu-latest' }}
       run: |
-        ./mvnw clean verify -DtrimStackTrace=false
+        ./mvnw clean verify
 
     - name: Authenticate to GCS
       if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/main' && matrix.os == 'ubuntu-latest' }}

CHANGELOG.md

@@ -2,11 +2,6 @@
 
 ## [2.1.0] - UNRELEASED
 ### Changes
-- add base URL preference for CLI download
-- use LS embedded in CLI and only download CLI as dependency
-
-## [2.1.0] - v20230720.102331
-### Changes
 - fix custom path configuration
 
 ## [2.1.0] - v20230606.182718

plugin/plugin.xml

@@ -167,7 +167,7 @@
       clientImpl="io.snyk.languageserver.protocolextension.SnykExtendedLanguageClient"
       lastDocumentDisconnectedTimeout="3000000"
       singleton="true"
-      markerType="io.snyk.languageserver.marker">
+      makerType="io.snyk.languageserver.marker">
     </server>
 
     <!-- snyk code -->

plugin/src/main/java/io/snyk/eclipse/plugin/SnykStartup.java

@@ -50,34 +50,35 @@ public void earlyStartup() {
       logger = Platform.getLog(getClass());
     }
     runtimeEnvironment = new LsRuntimeEnvironment();
-    Job initJob = new Job("Downloading latest CLI release...") {
+    Job initJob = new Job("Downloading latest Language Server release...") {
       @Override
       protected IStatus run(IProgressMonitor monitor) {
-        try {
-          logger.info("LS: Checking for needed download");
-          if (downloadLS()) {
-            monitor.beginTask("Downloading CLI", 100);
-            logger.info("LS: Need to download");
-            downloading = true;
-            download(monitor);
+        PlatformUI.getWorkbench().getDisplay().asyncExec(() -> {
+          try {
+            logger.info("LS: Checking for needed download");
+            if (downloadLS()) {
+              logger.info("LS: Need to download");
+              downloading = true;
+              monitor.subTask("Starting download of Snyk Language Server");
+              download(monitor);
+            }
+          } catch (Exception exception) {
+            logError(exception);
           }
-        } catch (Exception exception) {
-          logError(exception);
-        }
-        downloading = false;
-        monitor.subTask("Starting Snyk CLI in Language Server mode...");
-        startLanguageServer();
+          downloading = false;
+
+          monitor.subTask("Starting Snyk Language Server...");
+          startLanguageServer();
 
-        PlatformUI.getWorkbench().getDisplay().syncExec(() -> {
           if (Preferences.getInstance().getAuthToken().isBlank()) {
             monitor.subTask("Starting Snyk Wizard to configure initial settings...");
             SnykWizard wizard = new SnykWizard();
             WizardDialog dialog = new WizardDialog(PlatformUI.getWorkbench().getDisplay().getActiveShell(), wizard);
             dialog.setBlockOnOpen(true);
             dialog.open();
           }
+
         });
-        monitor.done();
 
         return Status.OK_STATUS;
       }
@@ -116,7 +117,7 @@ public static SnykView getSnykView() {
   }
 
   private boolean downloadLS() {
-    File lsFile = new File(Preferences.getInstance().getCliPath());
+    File lsFile = new File(Preferences.getInstance().getLsBinary());
     logger.info("LS: Expecting file at " + lsFile.getAbsolutePath());
     if (!Preferences.getInstance().isManagedBinaries()) {
       logger.info("LS: Managed binaries disabled, skipping download");
@@ -147,7 +148,7 @@ LsDownloader getLsDownloader() throws URISyntaxException {
 
   @SuppressWarnings("ResultOfMethodCallIgnored")
   IStatus download(IProgressMonitor monitor) {
-    final File lsFile = new File(Preferences.getInstance().getCliPath());
+    final File lsFile = new File(Preferences.getInstance().getLsBinary());
     try {
       LsDownloader lsDownloader = getLsDownloader();
       lsFile.getParentFile().mkdirs();

plugin/src/main/java/io/snyk/eclipse/plugin/properties/PreferencesPage.java

@@ -26,22 +26,22 @@ public PreferencesPage() {
 
 	@Override
 	public void init(IWorkbench workbench) {
-		setPreferenceStore(Preferences.getInstance().getStore());
+		setPreferenceStore(io.snyk.eclipse.plugin.properties.preferences.Preferences.getInstance().getStore());
 		setMessage("Snyk Preferences");
 	}
 
 	@Override
 	protected void createFieldEditors() {
 		TokenFieldEditor tokenField = new TokenFieldEditor(
-				Preferences.getInstance(),
-				Preferences.AUTH_TOKEN_KEY, "Snyk API Token:",
+				io.snyk.eclipse.plugin.properties.preferences.Preferences.getInstance(),
+				io.snyk.eclipse.plugin.properties.preferences.Preferences.AUTH_TOKEN_KEY, "Snyk API Token:",
 				getFieldEditorParent());
 		addField(tokenField);
-		addField(new StringFieldEditor(Preferences.PATH_KEY, "Path:",
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.PATH_KEY, "Path:",
 				getFieldEditorParent()));
-		addField(new StringFieldEditor(Preferences.ENDPOINT_KEY,
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.ENDPOINT_KEY,
 				"Custom Endpoint:", getFieldEditorParent()));
-		addField(new BooleanFieldEditor(Preferences.INSECURE_KEY,
+		addField(new BooleanFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.INSECURE_KEY,
 				"Allow unknown certificate authorities", getFieldEditorParent()));
 
 		addField(space());
@@ -52,23 +52,23 @@ protected void createFieldEditors() {
 				getFieldEditorParent()));
 		addField(space());
 		addField(new BooleanFieldEditor(
-				Preferences.ACTIVATE_SNYK_OPEN_SOURCE,
+				io.snyk.eclipse.plugin.properties.preferences.Preferences.ACTIVATE_SNYK_OPEN_SOURCE,
 				"Snyk Open Source enabled", getFieldEditorParent()));
 		snykCodeCheckbox = new BooleanFieldEditor(
-				Preferences.ACTIVATE_SNYK_CODE, "Snyk Code enable" + "d",
+				io.snyk.eclipse.plugin.properties.preferences.Preferences.ACTIVATE_SNYK_CODE, "Snyk Code enable" + "d",
 				getFieldEditorParent());
 
 		addField(snykCodeCheckbox);
-		addField(new BooleanFieldEditor(Preferences.ACTIVATE_SNYK_IAC,
+		addField(new BooleanFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.ACTIVATE_SNYK_IAC,
 				"Snyk Infrastructure-as-Code enabled", getFieldEditorParent()));
 
 		addField(space());
 		addField(new LabelFieldEditor("Advanced options:", getFieldEditorParent()));
-		addField(new StringFieldEditor(Preferences.ORGANIZATION_KEY,
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.ORGANIZATION_KEY,
 				"Organization:", getFieldEditorParent()));
-		addField(new StringFieldEditor(Preferences.ADDITIONAL_PARAMETERS,
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.ADDITIONAL_PARAMETERS,
 				"Additional Parameters:", getFieldEditorParent()));
-		addField(new StringFieldEditor(Preferences.ADDITIONAL_ENVIRONMENT,
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.ADDITIONAL_ENVIRONMENT,
 				"Additional Environment:", getFieldEditorParent()));
 
 		addField(space());
@@ -78,14 +78,14 @@ protected void createFieldEditors() {
 			System.out.println("managed bionaries changed");
 		});
 		addField(manageBinaries);
-		addField(new StringFieldEditor(Preferences.CLI_BASE_URL, "Base URL for CLI download:",
-            getFieldEditorParent()));
-		addField(new FileFieldEditor(Preferences.CLI_PATH, "Snyk CLI (incl. Language Server):",
+		addField(new FileFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.LS_BINARY_KEY,
+				"Snyk Language Server:", getFieldEditorParent()));
+		addField(new FileFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.CLI_PATH, "Snyk CLI:",
 				getFieldEditorParent()));
 
 		addField(space());
 
-		addField(new BooleanFieldEditor(Preferences.SEND_ERROR_REPORTS,
+		addField(new BooleanFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.SEND_ERROR_REPORTS,
 				"Send error reports to Snyk", getFieldEditorParent()));
 		addField(new BooleanFieldEditor(Preferences.ENABLE_TELEMETRY, "Send usage statistics to Snyk",
 				getFieldEditorParent()));
@@ -97,7 +97,7 @@ protected void createFieldEditors() {
 				+ "paths are safe to scan. Every path below a given path is considered safe to scan. \n"
 				+ "Please separate entries with \"" + File.pathSeparator + "\".",
 				getFieldEditorParent()));
-		addField(new StringFieldEditor(Preferences.TRUSTED_FOLDERS,
+		addField(new StringFieldEditor(io.snyk.eclipse.plugin.properties.preferences.Preferences.TRUSTED_FOLDERS,
 				"Trusted Folders:", getFieldEditorParent()));
 
 		disableSnykCodeIfOrgDisabled();

plugin/src/main/java/io/snyk/eclipse/plugin/properties/preferences/Preferences.java

@@ -4,7 +4,6 @@
 import org.eclipse.jface.preference.IPreferenceStore;
 
 import io.snyk.eclipse.plugin.EnvironmentConstants;
-import io.snyk.languageserver.LsRuntimeEnvironment;
 
 import java.io.File;
 import java.util.Optional;
@@ -13,11 +12,10 @@
 
 public class Preferences {
   static Preferences CURRENT_PREFERENCES;
-  static LsRuntimeEnvironment LS_RUNTIME_ENV = new LsRuntimeEnvironment();
 
   public static synchronized Preferences getInstance() {
     if (CURRENT_PREFERENCES == null) {
-      CURRENT_PREFERENCES = new Preferences(new SecurePreferenceStore()); 
+      CURRENT_PREFERENCES = new Preferences(new SecurePreferenceStore());
     }
     return CURRENT_PREFERENCES;
   }
@@ -33,8 +31,8 @@ public static synchronized Preferences getInstance(PreferenceStore store) {
   public static final String PATH_KEY = "path";
   public static final String ENDPOINT_KEY = "endpoint";
   public static final String INSECURE_KEY = "insecure";
+  public static final String LS_BINARY_KEY = "ls-binary";
   public static final String CLI_PATH = "cli-path";
-  public static final String CLI_BASE_URL = "cli-base-url";
   public static final String ACTIVATE_SNYK_CODE = "ACTIVATE_SNYK_CODE";
   public static final String ACTIVATE_SNYK_OPEN_SOURCE = "ACTIVATE_SNYK_OPEN_SOURCE";
   public static final String ACTIVATE_SNYK_IAC = "ACTIVATE_SNYK_IAC";
@@ -84,34 +82,34 @@ public static synchronized Preferences getInstance(PreferenceStore store) {
     if (getPref(AUTHENTICATION_METHOD) == null || getPref(AUTHENTICATION_METHOD).isBlank()) {
       store(AUTHENTICATION_METHOD, AUTH_METHOD_TOKEN);
     }
+    
+    if (getPref(LS_BINARY_KEY) == null || getPref(LS_BINARY_KEY).equals("")) {
+      store(LS_BINARY_KEY, getDefaultLsPath());
+    }
 
     String token = SystemUtils.getEnvironmentVariable(EnvironmentConstants.ENV_SNYK_TOKEN, "");
     if (getPref(AUTH_TOKEN_KEY) == null && !"".equals(token)) {
       store(AUTH_TOKEN_KEY, token);
     }
-    
     String endpoint = SystemUtils.getEnvironmentVariable(EnvironmentConstants.ENV_SNYK_API, "");
     if (getPref(ENDPOINT_KEY) == null && !"".equals(endpoint)) {
       store(ENDPOINT_KEY, endpoint);
     }
-    
     String org = SystemUtils.getEnvironmentVariable(EnvironmentConstants.ENV_SNYK_ORG, "");
     if (getPref(ORGANIZATION_KEY) == null && !"".equals(org)) {
       store(ORGANIZATION_KEY, org);
     }
-    
-    String cliPath = getDefaultCliPath();
-    if (getPref(CLI_PATH) == null && !"".equals(cliPath)) {
-      store(CLI_PATH, cliPath);
-    }
-    
-    if (getPref(CLI_BASE_URL) == null || getPref(CLI_BASE_URL).isBlank()) {
-      store(CLI_BASE_URL, "https://static.snyk.io");
-    }
   }
 
-  private String getDefaultCliPath() {
-    File binary = new File(getBinaryDirectory(), LS_RUNTIME_ENV.getDownloadBinaryName());
+  private String getBinaryName() {
+    var osName = SystemUtils.OS_NAME;
+    var executable = "snyk-ls";
+    if (osName.toLowerCase().startsWith("win")) executable += ".exe";
+    return executable;
+  }
+
+  private String getDefaultLsPath() {
+    File binary = new File(getBinaryDirectory(), getBinaryName());
     return binary.getAbsolutePath();
   }
 
@@ -131,6 +129,9 @@ public String getEndpoint() {
     return getPref(ENDPOINT_KEY, "");
   }
 
+  public String getLsBinary() {
+    return getPref(LS_BINARY_KEY, "");
+  }
   public String getLspVersion() {
     return getPref(LSP_VERSION);
   }
@@ -144,7 +145,7 @@ public Optional<String> getPath() {
   }
 
   public String getCliPath() {
-    return getPref(CLI_PATH, getDefaultCliPath());
+    return getPref(CLI_PATH, "");
   }
 
   public boolean isInsecure() {

plugin/src/main/java/io/snyk/eclipse/plugin/utils/FileDownloadResponseHandler.java

@@ -14,6 +14,7 @@ public class FileDownloadResponseHandler implements ResponseHandler<File> {
 
   private final File destinationFile;
   private final IProgressMonitor progressMonitor;
+  private static final String HEADER_CONTENT_LENGTH = "content-length";
 
   public FileDownloadResponseHandler(File file, IProgressMonitor monitor) {
     this.destinationFile = file;

plugin/src/main/java/io/snyk/eclipse/plugin/utils/LsMetadataResponseHandler.java

@@ -1,27 +1,25 @@
 package io.snyk.eclipse.plugin.utils;
 
-import java.io.IOException;
-import java.io.InputStream;
-import java.nio.charset.Charset;
-
+import com.fasterxml.jackson.databind.ObjectMapper;
+import io.snyk.languageserver.download.LsMetadata;
 import org.apache.http.HttpResponse;
 import org.apache.http.client.ResponseHandler;
 
-public class LsMetadataResponseHandler implements ResponseHandler<String> {
+import java.io.IOException;
+import java.io.InputStream;
+
+public class LsMetadataResponseHandler implements ResponseHandler<LsMetadata> {
+  private final ObjectMapper om = new ObjectMapper();
 
   public LsMetadataResponseHandler() {
   }
 
   @Override
-  public String handleResponse(HttpResponse httpResponse) {
-    String latestCliSupportingLSProtocolVersion = "10";
-    InputStream inputStream;
-    try {
-      inputStream = httpResponse.getEntity().getContent();
-      latestCliSupportingLSProtocolVersion = new String(inputStream.readAllBytes(), Charset.defaultCharset());
+  public LsMetadata handleResponse(HttpResponse httpResponse) {
+    try (InputStream inputStream = httpResponse.getEntity().getContent()) {
+      return om.readValue(inputStream, LsMetadata.class);
     } catch (UnsupportedOperationException | IOException e) {
       throw new RuntimeException(e);
     }
-    return latestCliSupportingLSProtocolVersion;
   }
 }

plugin/src/main/java/io/snyk/languageserver/LsRuntimeEnvironment.java

@@ -24,31 +24,42 @@ public class LsRuntimeEnvironment {
   public static final Map<String, String> map = new HashMap<>();
 
   static {
-    map.put("win", "win");
+    map.put("win", "windows");
     map.put("lin", "linux");
-    map.put("mac", "macos");
-    map.put("dar", "macos");
-    map.put("macosx", "macos");
-
-    map.put("amd64", "");
-    map.put("x8664", "");
-    map.put("x86_64", "");
-    map.put("x64", "");
-
-    map.put("aarch_64", "-arm64");
-    map.put("aarch64", "-arm64");
-    map.put("arm64", "-arm64");
+    map.put("mac", "darwin");
+    map.put("dar", "darwin");
+    map.put("macosx", "darwin");
+
+    map.put("amd64", "amd64");
+    map.put("x8664", "amd64");
+    map.put("x86_64", "amd64");
+    map.put("x64", "amd64");
+
+    map.put("aarch_64", "arm64");
+    map.put("aarch64", "arm64");
+    map.put("arm64", "arm64");
+
+    map.put("arm_32", "arm");
+    map.put("arm32", "arm");
+    map.put("arm", "arm");
+
+    map.put("x8632", "386");
+    map.put("x86_32", "386");
+    map.put("x86", "386");
+    map.put("ia32", "386");
+    map.put("486", "386");
+    map.put("586", "386");
+    map.put("686", "386");
   }
 
   public LsRuntimeEnvironment() {
   }
 
-  public String getDownloadBinaryName() {
-    String base = "snyk-%s%s";
+  public String getDownloadBinaryName(String version) {
+    String base = "snyk-ls_%s_%s_%s";
     String os = getOs();
-    String executable = String.format(base, os, getArch());
-    if (executable.toLowerCase().contains("win"))
-      executable += ".exe";
+    String executable = String.format(base, version, os, getArch());
+    if (executable.toLowerCase().contains("windows")) executable += ".exe";
     return executable;
   }