rescue error when invalid env config is passes from client

Author: lukaszsamson

apps/elixir_ls_debugger/lib/debugger/server.ex

@@ -1702,7 +1702,19 @@ defmodule ElixirLS.Debugger.Server do
   end
 
   defp set_env_vars(env) when is_map(env) do
-    for {k, v} <- env, do: System.put_env(k, v)
+    try do
+      System.put_env(env)
+    rescue
+      e ->
+        Output.debugger_console(
+          "Cannot set environment variables to #{inspect(env)}: #{Exception.message(e)}"
+        )
+
+        Output.debugger_important(
+          "Invalid `env` in launch configuration. Expected a map with string key value pairs, got #{inspect(env)}."
+        )
+    end
+
     :ok
   end
 

apps/language_server/lib/language_server/server.ex

@@ -1640,7 +1640,19 @@ defmodule ElixirLS.LanguageServer.Server do
     prev_env = state.settings["envVariables"]
 
     if is_nil(prev_env) or env == prev_env do
-      System.put_env(env)
+      try do
+        System.put_env(env)
+      rescue
+        e ->
+          Logger.error(
+            "Cannot set environment variables to #{inspect(env)}: #{Exception.message(e)}"
+          )
+
+          JsonRpc.show_message(
+            :error,
+            "Invalid `envVariables` in configuration. Expected a map with string key value pairs, got #{inspect(env)}."
+          )
+      end
     else
       JsonRpc.show_message(
         :warning,