Require / prefix on URL paths in Python (#1457)

Author: eddyashton

python/ccf/clients.py

@@ -533,7 +533,7 @@ def call(
         """
         Issues one request, synchronously, and returns the response.
 
-        :param str path: URI of the targeted resource.
+        :param str path: URI of the targeted resource. Must begin with '/'
         :param dict body: Request body (optional).
         :param str http_verb: HTTP verb (e.g. "POST" or "GET").
         :param dict headers: HTTP request headers (optional).
@@ -542,6 +542,9 @@ def call(
 
         :return: :py:class:`ccf.clients.Response`
         """
+        if not path.startswith("/"):
+            raise ValueError(f"URL path '{path}' is invalid, must start with /")
+
         if self.is_connected:
             return self._direct_call(path, body, http_verb, headers, signed, timeout)
 

tests/suite/test_requirements.py

@@ -92,7 +92,7 @@ def check(network, args, *nargs, **kwargs):
         primary, _ = network.find_primary()
         with primary.client("member0") as c:
             r = c.post(
-                "gov/query",
+                "/gov/query",
                 {
                     "text": """tables = ...
                         trusted_nodes_count = 0