Merge ~lloydwaltersj/maas:fix-broken-oapi-param-names into maas:master

[Jack Lloyd-Walters]

fix LP:2009140, OpenApi parameter names erroneously stripped of trailing '}'

[MAAS Lander (maas-lander)]

UNIT TESTS
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/2057/consoleText
COMMIT: 9f11adbcb6307054be03f1f1928e801c931420c5

review: Needs Fixing

[MAAS Lander (maas-lander)]

UNIT TESTS
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/2058/consoleText
COMMIT: d71a82dc918fd6ee2f2d5d6086cfda580d954ea7

review: Needs Fixing

[Jack Lloyd-Walters (lloydwaltersj)]

jenkins: !test

[Alberto Donato (ack)]

+1

small nit inline

review: Approve

[MAAS Lander (maas-lander)]

UNIT TESTS
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED
LOG: http://maas-ci.internal:8080/job/maas-tester/2060/consoleText
COMMIT: d71a82dc918fd6ee2f2d5d6086cfda580d954ea7

review: Needs Fixing

[Jack Lloyd-Walters (lloydwaltersj)]

jenkins: !test

[MAAS Lander (maas-lander)]

UNIT TESTS
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 3ce2d282c36063991e8ea2abb14cbe2eb26396c0

review: Approve

[MAAS Lander (maas-lander)]

LANDING
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED BUILD
LOG: http://maas-ci.internal:8080/job/maas-tester/2062/consoleText

[Adam Collard (adam-collard)]

jenkins: !test

[MAAS Lander (maas-lander)]

UNIT TESTS
-b fix-broken-oapi-param-names lp:~lloydwaltersj/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 8c7091939380f65768da6310d689ca7212857fa8

review: Approve

diff --git a/src/maasserver/api/doc_oapi.py b/src/maasserver/api/doc_oapi.py
index b7e3a9d..fba7926 100644
--- a/src/maasserver/api/doc_oapi.py
+++ b/src/maasserver/api/doc_oapi.py
@@ -22,6 +22,11 @@ from maasserver.djangosettings import settings
 from maasserver.models.config import Config
 from maasserver.utils import build_absolute_uri
 
+# LP 2009140: Match a par of brackets enclosing a string, return only the string within the brackets.
+# Functions as a more context aware equivalent to string.strip("}{")
+# ie: '{param}' returns 'param', 'param_{test}' is unnafected
+PARAM_RE = re.compile(r"(?<=\{)\.+(?<=\})")
+
 
 def landing_page(request):
     """Render a landing page with pointers for the MAAS API.
@@ -213,8 +218,11 @@ def _oapi_item_from_docstring(
             oper_obj["deprecated"] = True
         for param in ap_dict["params"]:
             description = _prettify(param["description_stripped"])
-            name = param["name"].strip("}{")
-            path_var = name in uri_params
+            # LP 2009140
+            stripped_name = PARAM_RE.match(param["name"])
+            name = (
+                param["name"] if not stripped_name else stripped_name.group()
+            )
             required = (
                 param["options"]["required"].lower() == "true"
                 or name != param["name"]
@@ -223,7 +231,7 @@ def _oapi_item_from_docstring(
             if name != param["name"]:
                 param_dict = {
                     "name": name,
-                    "in": "path" if path_var else "query",
+                    "in": "path" if name in uri_params else "query",
                     "description": description,
                     "schema": {
                         "type": _type_to_string(param["type"]),