(whimsy) branch master updated: System maintained lists must be OK

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 9ad0b6c6 System maintained lists must be OK
9ad0b6c6 is described below

commit 9ad0b6c6b70b51c2ca5c04166fad31926b6d0f94
Author: Sebb <se...@apache.org>
AuthorDate: Sun Sep 15 13:31:44 2024 +0100

    System maintained lists must be OK
---
 lib/whimsy/asf/mail.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/whimsy/asf/mail.rb b/lib/whimsy/asf/mail.rb
index ee8359a0..bbf8aa05 100644
--- a/lib/whimsy/asf/mail.rb
+++ b/lib/whimsy/asf/mail.rb
@@ -96,8 +96,8 @@ module ASF
       flags = getflags(listid)
       return nil if flags.nil? # Not a known list
       return true unless isModSub?(flags) # subscription not needed
-      return true if listid == 'committers@apache.org' # any more like this?
-      return true if self._committers_allowed().include?(listid) 
+      return true if self._cannot_unsub.include? listid # must be system maintained, so assume OK
+      return true if self._committers_allowed().include?(listid)
       return true if member # They can read anything
       return true if pmc_chair and self._chairs_allowed.include? listid
       return true if ldap_pmcs and ldap_pmcs.include? listid.split('@')[-1].sub('.apache.org', '')