[whimsy] branch master updated: defensive programming: validate podling name

[ru...@apache.org]

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

rubys pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/whimsy.git

The following commit(s) were added to refs/heads/master by this push:
       new  3c81844   defensive programming: validate podling name
3c81844 is described below

commit 3c81844263c8642bd8ac483507aebe5e16ad28c5
Author: Sam Ruby <ru...@intertwingly.net>
AuthorDate: Wed Aug 3 10:30:26 2016 -0400

    defensive programming: validate podling name
---
 www/secretary/workbench/file.cgi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/secretary/workbench/file.cgi b/www/secretary/workbench/file.cgi
index 557f314..3f4d2bc 100755
--- a/www/secretary/workbench/file.cgi
+++ b/www/secretary/workbench/file.cgi
@@ -287,7 +287,7 @@ def send_email(target, message)
       cc << vars.email_cc if vars.email_cc
 
       # calculate podling list name
-      if vars.podling 
+      if vars.podling and vars.podling =~ /^\w[-\w]+$/
         if OLDPODLINGS.include?(vars.podling) 
           vars.podlinglist = "#{vars.podling}-private@incubator.apache.org"
         else
@@ -297,7 +297,7 @@ def send_email(target, message)
 
       # add pmc and podling lists, if supplied
       cc << "private@#{vars.pmc}.apache.org" if vars.pmc
-      cc << "#{vars.podlinglist}" if vars.podling
+      cc << "#{vars.podlinglist}" if vars.podlinglist
       cc << "private@incubator.apache.org" if vars.podling and not vars.pmc
 
       # replace the list of cc's

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <co...@whimsical.apache.org>'].