Removed a print that looked like a debug statement, as it was showing

up before the Content-type: line.  The first time someone tried to
change an option on a list (and only the first time), there'd be an
Internal Server Error.

1 files changed, 17 insertions, 2 deletions

diff --git a/cgi/admin b/cgi/admin
index 31640ef87..c097f7b60 100755
--- a/cgi/admin
+++ b/cgi/admin
@@ -21,7 +21,7 @@
 To run stand-alone for debugging, set env var PATH_INFO to name of list
 and, optionally, options category."""
 
-__version__ = "$Revision: 690 $"
+__version__ = "$Revision: 694 $"
 
 import sys
 import os, cgi, string, crypt, types, time
@@ -84,6 +84,22 @@ def isAuthenticated(list, password=None, SECRET="SECRET"):
         except mm_err.MMBadPasswordError:
             AddErrorMessage(doc, 'Error: Incorrect admin password.')
             return 0
+        except:
+	  print "Content-type: text/html\n"
+
+	  print "<p><h3>We're sorry, we hit a bug!</h3>\n"
+	  print "If you would like to help us identify the problem, please "
+	  print "email a copy of this page to the webmaster for this site"
+	  print 'with a description of what happened.  Thanks!'
+	  print "\n<PRE>"
+	  try:
+	    import traceback
+	    sys.stderr = sys.stdout
+	    traceback.print_exc()
+	  except:
+	    print "[failed to get traceback]"
+	  print "\n\n</PRE>"
+
         token = md5.new(SECRET + list_name + SECRET).digest()
         token = base64.encodestring(token)
         token = string.strip(token)
@@ -724,7 +740,6 @@ def ChangeOptions(lst, category, cgi_info, document):
                 val = cgi_info[property].value
             value = GetValidValue(lst, property, kind, val, deps)
             if getattr(lst, property) != value:
-                print "property: ", property, "value: ", value
                 setattr(lst, property, value)
                 dirty = 1
     #