This is an automated email from the ASF dual-hosted git repository.
sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-release.git
The following commit(s) were added to refs/heads/main by this push:
new 65be23f Fix a bug with getting committees from ASFQuart user sessions
65be23f is described below
commit 65be23fe73ddce65411ab825a2a32f82d741d143
Author: Sean B. Palmer <[email protected]>
AuthorDate: Fri Sep 19 14:22:35 2025 +0100
Fix a bug with getting committees from ASFQuart user sessions
---
atr/principal.py | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/atr/principal.py b/atr/principal.py
index e7da584..8437567 100644
--- a/atr/principal.py
+++ b/atr/principal.py
@@ -252,7 +252,15 @@ class AuthoriserASFQuart:
return
# We do not check that the ASF UID is the same as the one in the
session
# It is the caller's responsibility to ensure this
- self.__cache.member_of[asf_uid] =
frozenset(asfquart_session.get("pmcs", []))
+ self.__cache.member_of[asf_uid] =
frozenset(asfquart_session.get("committees", []))
+
+ # TODO: The code previously used "pmcs" above, which was incorrect
+ # The following is a defense in depth check until the reason for this
is investigated
+ # If there is no evidence to the contrary, we can assume that it was a
simple typo
+ if ("pmcs" in asfquart_session) and ("committees" not in
asfquart_session):
+ asfquart_session["committees"] = asfquart_session["pmcs"]
+ # End of defense in depth check
+
self.__cache.participant_of[asf_uid] =
frozenset(asfquart_session.get("projects", []))
self.__cache.last_refreshed = int(time.time())
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
