fix(shell): use _ instead of - to sanitize environment variable n…

…ame (#160) Close #160
inetum-orleans · Jan 14, 2021 · f9e11a8 · f9e11a8
1 parent 6f4f938
commit f9e11a8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ddb/feature/shell/integrations.py b/ddb/feature/shell/integrations.py
@@ -118,7 +118,7 @@ def __init__(self):
 
     @staticmethod
     def _sanitize_key(key):
-        return slugify(key, regex_pattern=r'[^-a-zA-Z0-9_]+').upper()
+        return slugify(key, regex_pattern=r'[^-a-zA-Z0-9_]+', separator="_").upper()
 
     def set_environment_variable(self, key, value):
         yield "export " + self._sanitize_key(key) + "=" + shlex.quote(value)
@@ -194,7 +194,7 @@ def __init__(self):
 
     @staticmethod
     def _sanitize_key(key):
-        return slugify(key, regex_pattern=r'[^-a-zA-Z0-9_]+').upper()
+        return slugify(key, regex_pattern=r'[^-a-zA-Z0-9_]+', separator="_").upper()
 
     def set_environment_variable(self, key, value):
         # TODO: Maybe use subprocess.list2cmdline for Windows ?