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 20828d3 Don't check entries not in iclas.txt
20828d3 is described below
commit 20828d360cdb14285d26cb080f5e72aa145c72ef
Author: Sebb <[email protected]>
AuthorDate: Mon Jun 21 23:03:04 2021 +0100
Don't check entries not in iclas.txt
---
www/members/ldap-namecheck.cgi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/www/members/ldap-namecheck.cgi b/www/members/ldap-namecheck.cgi
index 7a27e5d..632f007 100755
--- a/www/members/ldap-namecheck.cgi
+++ b/www/members/ldap-namecheck.cgi
@@ -82,7 +82,8 @@ _html do
badSN += 1 unless snOK
icla = ASF::ICLA.find_by_id(p.uid)
- public_name = icla.name rescue '?'
+ public_name = icla.name rescue nil
+ next unless public_name # Don't check entries not in iclas.txt
cnOK = (public_name == p.cn)
pnames=public_name.split
