impr: Clarify comment on (sometimes) optional SSO config values

element-hq · Nov 27, 2024 · b506406 · b506406
1 parent a38b58c
commit b506406
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/synapse/config/sso.py b/synapse/config/sso.py
@@ -43,7 +43,8 @@ class SsoAttributeRequirement:
     """Object describing a single requirement for SSO attributes."""
 
     attribute: str
-    # If a value is not given, than the attribute must simply exist.
+    # If neither value nor one_of is given, the attribute must simply exist. This is
+    # only true for CAS configs which use a different JSON schema than the one below.
     value: Optional[str] = None
     one_of: Optional[List[str]] = None