Schema for authn_request ui_locales is not correct italia#303

pietrodematteis · Jan 23, 2024 · 03dbb9c · 03dbb9c
1 parent 1faa95e
commit 03dbb9c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/spid_cie_oidc/provider/schemas/authn_requests.py b/spid_cie_oidc/provider/schemas/authn_requests.py
@@ -133,7 +133,7 @@ class AuthenticationRequest(BaseModel):
     claims: Optional[dict]
     state: constr(min_length=32)
     # TODO: to be improved
-    ui_locales: Optional[List[str]]
+    ui_locales: Optional[constr(regex=r'[^ ]+(\s([^ ]+))*')]
 
     # sub claim MUST not be used to prevent that this jwt
     # could be reused as a private_key_jwt
@@ -186,7 +186,7 @@ def example():  # pragma: no cover
             acr_values= ["https://www.spid.gov.it/SpidL2", "https://www.spid.gov.it/SpidL1"],
             claims= {},
             state= "fyZiOL9Lf2CeKuNT2JzxiLRDink0uPcd",
-            ui_locales= ["codice1", "codice2", "codice3"],
+            ui_locales= "codice1 codice2 codice3",
             sub= "https://rp.cie.it/",
             iss= "https://op.spid.agid.gov.it/",
             aud= ["https://rp.spid.agid.gov.it/auth"],

diff --git a/spid_cie_oidc/provider/tests/authn_request_settings.py b/spid_cie_oidc/provider/tests/authn_request_settings.py
@@ -39,7 +39,7 @@
     "acr_values": ["https://www.spid.gov.it/SpidL2", "https://www.spid.gov.it/SpidL1"],
     "claims": CLAIMS_SPID,
     "state": "fyZiOL9Lf2CeKuNT2JzxiLRDink0uPcd",
-    "ui_locales": ["codice1", "codice2", "codice3"],
+    "ui_locales": "codice1 codice2 codice3",
     "sub": "https://rp.cie.it/",
     "iss": "https://op.spid.agid.gov.it/",
     "aud": ["https://rp.spid.agid.gov.it/auth"],