Merge pull request #2117 from python-gitlab/jlvillal/encodedid_path

fix: ensure path elements are escaped

2 files changed, 7 insertions, 5 deletions

diff --git a/gitlab/base.py b/gitlab/base.py
index 205a0f2..76524d1 100644
--- a/gitlab/base.py
+++ b/gitlab/base.py
@@ -380,10 +380,12 @@ class RESTManager:
         if self._parent is None or not self._from_parent_attrs:
             return path
 
-        data = {
-            self_attr: getattr(self._parent, parent_attr, None)
-            for self_attr, parent_attr in self._from_parent_attrs.items()
-        }
+        data: Dict[str, Optional[gitlab.utils.EncodedId]] = {}
+        for self_attr, parent_attr in self._from_parent_attrs.items():
+            if not hasattr(self._parent, parent_attr):
+                data[self_attr] = None
+                continue
+            data[self_attr] = gitlab.utils.EncodedId(getattr(self._parent, parent_attr))
         self._parent_attrs = data
         return path.format(**data)
 
diff --git a/tests/functional/conftest.py b/tests/functional/conftest.py
index 09cdd69..22457ba 100644
--- a/tests/functional/conftest.py
+++ b/tests/functional/conftest.py
@@ -391,7 +391,7 @@ def project_file(project):
 @pytest.fixture(scope="function")
 def release(project, project_file):
     _id = uuid.uuid4().hex
-    name = f"test-release-{_id}"
+    name = f"we_have_a_slash/test-release-{_id}"
 
     project.refresh()  # Gets us the current default branch
     release = project.releases.create(