Rename environment.get_value_string().

Author: letitz

src/clusterfuzz/_internal/bot/tasks/utasks/fuzz_task.py

@@ -1077,11 +1077,10 @@ def key_fn(crash):
 
 def _get_issue_metadata_from_environment(variable_name):
   """Get issue metadata from environment."""
-  values = str(environment.get_value_string(variable_name, '')).split(',')
+  values = environment.get_value_raw(variable_name, '').split(',')
   # Allow a variation with a '_1' to specified. This is needed in cases where
   # this is specified in both the job and the bot environment.
-  values.extend(
-      str(environment.get_value_string(variable_name + '_1', '')).split(','))
+  values.extend(environment.get_value_raw(variable_name + '_1', '').split(','))
   return [value.strip() for value in values if value.strip()]
 
 

src/clusterfuzz/_internal/system/environment.py

@@ -596,9 +596,9 @@ def get_ubsan_disabled_options():
   }
 
 
-def get_value_string(environment_variable, default_value=None):
-  """Get environment variable (as a string)."""
-  return os.getenv(environment_variable, default_value)
+def get_value_raw(var: str, default: Optional[str] = None) -> Optional[str]:
+  """Returns environment variable `var` directly, without evaluating it."""
+  return os.environ.get(var, default)
 
 
 def get_value(environment_variable, default_value=None, env=None):