Skip to content

Commit 3fcd4ea

Browse files
mabeziMarkus Ziehe
authored andcommitted
Adjust regex pattern to upstream & remove useless character conversion
1 parent 02f8839 commit 3fcd4ea

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

plugins/module_utils/netbox_secrets.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,8 @@ def run(self):
3636
# Used for msg output
3737
name = data.get("name")
3838

39-
data["slug"] = self._to_slug(name)
39+
if not data.get("slug"):
40+
data["slug"] = self._to_slug(name)
4041

4142
object_query_params = self._build_query_params(
4243
endpoint_name, data, user_query_params

plugins/module_utils/netbox_utils.py

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1350,7 +1350,7 @@ def _find_ids(self, data, user_query_params):
13501350
"tenants",
13511351
"tags",
13521352
) and isinstance(list_item, str):
1353-
temp_dict = {"slug": self._to_slug(list_item)}
1353+
temp_dict = {"slug": list_item}
13541354
elif isinstance(list_item, dict):
13551355
norm_data = self._normalize_data(list_item)
13561356
temp_dict = self._build_query_params(
@@ -1423,10 +1423,11 @@ def _to_slug(self, value):
14231423
return value
14241424
elif isinstance(value, int):
14251425
return value
1426-
else:
1427-
removed_chars = re.sub(r"[^\-\.\w\s]", "", value)
1428-
convert_chars = re.sub(r"[\-\.\s]", "-", removed_chars)
1429-
return convert_chars.strip().lower()
1426+
1427+
value = re.sub(r"[^\-.\w\s]", "", value)
1428+
value = re.sub(r"^[\s.]+|[\s.]+$", "", value)
1429+
value = re.sub(r"[-.\s]+", "-", value)
1430+
return value.strip().lower()
14301431

14311432
def _normalize_data(self, data):
14321433
"""

0 commit comments

Comments
 (0)