From f9ca59c12991dd24532c876d71702bc724203826 Mon Sep 17 00:00:00 2001 From: Erik Sundell Date: Mon, 27 Nov 2023 18:30:30 +0100 Subject: [PATCH] cilogon: fix missing schema entry for default under allowed_idps --- oauthenticator/schemas/cilogon-schema.yaml | 2 ++ oauthenticator/tests/test_cilogon.py | 1 + 2 files changed, 3 insertions(+) diff --git a/oauthenticator/schemas/cilogon-schema.yaml b/oauthenticator/schemas/cilogon-schema.yaml index 3b97ce53..2f8436ff 100644 --- a/oauthenticator/schemas/cilogon-schema.yaml +++ b/oauthenticator/schemas/cilogon-schema.yaml @@ -15,6 +15,8 @@ properties: type: string allowed_domains_claim: type: string + default: + type: boolean username_derivation: type: object additionalProperties: false diff --git a/oauthenticator/tests/test_cilogon.py b/oauthenticator/tests/test_cilogon.py index d8f46c54..4dac4927 100644 --- a/oauthenticator/tests/test_cilogon.py +++ b/oauthenticator/tests/test_cilogon.py @@ -636,6 +636,7 @@ async def test_allowed_idps_username_derivation_actions(cilogon_client): c.CILogonOAuthenticator.allow_all = True c.CILogonOAuthenticator.allowed_idps = { 'https://strip-idp-domain.example.com/login/oauth/authorize': { + 'default': True, 'username_derivation': { 'username_claim': 'email', 'action': 'strip_idp_domain',