whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sebastian Bazley <s...@apache.org>
Subject [whimsy.git] [1/2] Commit cb09e26: Include? needs to be applied to a string to match a substring
Date Sat, 23 Jan 2016 12:53:00 GMT
Commit cb09e26f56eda7aa56a43db801757f9679204884:
    Include? needs to be applied to a string to match a substring


Branch: refs/heads/master
Author: Sebb <sebb@apache.org>
Committer: Sebb <sebb@apache.org>
Pusher: sebb <sebb@apache.org>

------------------------------------------------------------
lib/whimsy/asf/ldap.rb                                       | + -
------------------------------------------------------------
2 changes: 1 additions, 1 deletions.
------------------------------------------------------------


diff --git a/lib/whimsy/asf/ldap.rb b/lib/whimsy/asf/ldap.rb
index 9fda6e0..f8db23f 100644
--- a/lib/whimsy/asf/ldap.rb
+++ b/lib/whimsy/asf/ldap.rb
@@ -270,7 +270,7 @@ def asf_committer?
     end
 
     def banned?
-      not attrs['loginShell'] or attrs['loginShell'].include? "/usr/bin/false" or attrs['loginShell'].include?
"bin/nologin" or attrs['loginShell'].include? "bin/no-cla"
+      not attrs['loginShell'] or %w(/usr/bin/false bin/nologin bin/no-cla).any? {|a| attrs['loginShell'].first.include?
a}
     end
 
     def mail

Mime
View raw message