fix: remove unnecessary token related env variables (#134)

Co-authored-by: Peter Schäfer <101886095+PeterSchafer@users.noreply.github.com>
snyk · Apr 24, 2023 · 5214c76 · 5214c76
1 parent b51671a
commit 5214c76
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/plugin/src/main/java/io/snyk/eclipse/plugin/runner/ProcessRunner.java b/plugin/src/main/java/io/snyk/eclipse/plugin/runner/ProcessRunner.java
@@ -125,8 +125,11 @@ private void setupProcessBuilderBase(ProcessBuilder pb) {
     if (token != null && authMethod.equals(Preferences.AUTH_METHOD_OAUTH)) {      
       pb.environment().put(EnvironmentConstants.ENV_INTERNAL_SNYK_OAUTH_ENABLED, "1");
       pb.environment().put(EnvironmentConstants.ENV_INTERNAL_OAUTH_TOKEN_STORAGE, token);
+      pb.environment().remove(EnvironmentConstants.ENV_SNYK_TOKEN);
     } else {      
       pb.environment().put(EnvironmentConstants.ENV_SNYK_TOKEN, token);
+      pb.environment().remove(EnvironmentConstants.ENV_INTERNAL_OAUTH_TOKEN_STORAGE);
+
     }
 
     String insecure = Preferences.getInstance().getPref(Preferences.INSECURE_KEY);

diff --git a/plugin/src/main/java/io/snyk/languageserver/LsRuntimeEnvironment.java b/plugin/src/main/java/io/snyk/languageserver/LsRuntimeEnvironment.java
@@ -74,14 +74,6 @@ String getOs() {
   }
 
   public void updateEnvironment(Map<String, String> env) {
-    String authToken = Preferences.getInstance().getAuthToken();
-    if (authToken != null && !authToken.isBlank()) {
-      env.put("SNYK_TOKEN", authToken);
-    }
-    String endpoint = Preferences.getInstance().getEndpoint();
-    if (endpoint != null && !endpoint.isEmpty()) {
-      env.put("SNYK_API", endpoint);
-    }
     addPath(env);
     addIntegrationInfoToEnv(env);
     addProxyToEnv(env);