summaryrefslogtreecommitdiff
path: root/src/services/api
diff options
context:
space:
mode:
authoromnom62 <omnom62@outlook.com>2026-09-22 06:37:12 +1000
committeromnom62 <omnom62@outlook.com>2026-09-29 06:58:05 +1000
commitf75c7b74dc4c2f38ad9e61e08dfe9fd1fc0686f9 (patch)
tree93db5dffca052d4bec4ada58c730915bdf255efd /src/services/api
parented218c254d964775ddf24158ac6520972d575faf (diff)
downloadvyos-1x-f75c7b74dc4c2f38ad9e61e08dfe9fd1fc0686f9.tar.gz
vyos-1x-f75c7b74dc4c2f38ad9e61e08dfe9fd1fc0686f9.zip
https: T8989: Hotfix to enforce audience settings with OIDC is configured
Diffstat (limited to 'src/services/api')
-rw-r--r--src/services/api/rest/libs/token_auth.py14
1 files changed, 10 insertions, 4 deletions
diff --git a/src/services/api/rest/libs/token_auth.py b/src/services/api/rest/libs/token_auth.py
index e0710cc20..ccc5fbf35 100644
--- a/src/services/api/rest/libs/token_auth.py
+++ b/src/services/api/rest/libs/token_auth.py
@@ -56,10 +56,16 @@ def verify_oidc_token(token: str):
issuer=state.oidc_issuer,
leeway=30,
)
- if state.oidc_audience:
- decode_kwargs["audience"] = state.oidc_audience
- else:
- decode_options["verify_aud"] = False
+ if not state.oidc_audience:
+ # Fail closed: config-time validation in service_https.py's
+ # verify() requires oidc.audience whenever oidc.issuer is
+ # set, but never trust that alone for a security-critical
+ # check. PyJWT also skips aud validation whenever no
+ # "audience" kwarg is passed at all, independent of
+ # verify_aud -- so this must refuse outright, not just stop
+ # forcing verify_aud=False.
+ return None
+ decode_kwargs["audience"] = state.oidc_audience
payload = jwt.decode(token, signing_key.key, **decode_kwargs)
return payload.get("sub") or payload.get("client_id")
except Exception: