Skip to content

Commit 223132c

Browse files
authored
Trim comma delimited lists for context (#3790)
1 parent eea0801 commit 223132c

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

src/cfnlint/context/context.py

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,12 @@ def ref(self, context: Context) -> Iterator[Any]:
278278
pass
279279

280280

281+
def _strip(value: Any) -> Any:
282+
if isinstance(value, str):
283+
return value.strip()
284+
return value
285+
286+
281287
@dataclass
282288
class Parameter(_Ref):
283289
"""
@@ -321,14 +327,17 @@ def __post_init__(self, parameter) -> None:
321327
if "Default" in parameter:
322328
default = parameter.get("Default", "")
323329
if isinstance(default, str):
324-
self.default = default.split(",")
330+
self.default = [_strip(value) for value in default.split(",")]
325331
else:
326-
self.default = [default]
332+
self.default = [_strip(default)]
333+
327334
for allowed_value in parameter.get("AllowedValues", []):
328335
if isinstance(allowed_value, str):
329-
self.allowed_values.append(allowed_value.split(","))
336+
self.allowed_values.append(
337+
[_strip(value) for value in allowed_value.split(",")]
338+
)
330339
else:
331-
self.allowed_values.append([allowed_value])
340+
self.allowed_values.append([_strip(allowed_value)])
332341
else:
333342
self.default = parameter.get("Default")
334343
self.allowed_values = parameter.get("AllowedValues")

0 commit comments

Comments
 (0)