Skip to content

Commit ded9419

Browse files
patch issue with compose spec environemnt vars being a dict
1 parent e3e71e3 commit ded9419

File tree

1 file changed

+12
-5
lines changed
  • services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core

1 file changed

+12
-5
lines changed

services/dynamic-sidecar/src/simcore_service_dynamic_sidecar/core/validation.py

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -107,14 +107,21 @@ def _apply_templating_directives(
107107

108108

109109
def _merge_env_vars(
110-
compose_spec_env_vars: list[str], settings_env_vars: list[str]
110+
compose_spec_env_vars: list[str] | dict[str, str],
111+
settings_env_vars: list[str] | dict[str, str],
111112
) -> list[str]:
112113
def _gen_parts_env_vars(
113-
env_vars: list[str],
114+
env_vars: list[str] | dict[str, str],
114115
) -> Generator[tuple[str, str], None, None]:
115-
for env_var in env_vars:
116-
key, value = env_var.split("=")
117-
yield key, value
116+
assert isinstance(env_vars, list | dict) # nosec
117+
118+
if isinstance(env_vars, list):
119+
for env_var in env_vars:
120+
key, value = env_var.split("=")
121+
yield key, value
122+
else:
123+
for key, value in env_vars.items():
124+
yield key, value
118125

119126
# pylint: disable=unnecessary-comprehension
120127
dict_spec_env_vars = {k: v for k, v in _gen_parts_env_vars(compose_spec_env_vars)}

0 commit comments

Comments
 (0)