Error/crash in admindb - Solved

First of all, the bug I reported earlier happens if you access this URL:
http://host/mailman/admindb/ <-- note trailing slash
It tries to cause an "Invalid options to CGI script" error, but can't because of the typo in Mailman/Cgi/admindb.py, line 77:
doc.AddItem(eader(2, "Invalid options to CGI script."))
where Header is misspelled. That fixed the crash. But I think that this URL (admindb/) should work just like admindb without the slash -- it should say that you must specify a list.
-Brian Ryner bryner@uiuc.edu

"BR" == Brian Ryner <bryner@uiuc.edu> writes:
BR> First of all, the bug I reported earlier happens if you access
BR> this URL:
BR> http://host/mailman/admindb/ <-- note trailing slash
BR> It tries to cause an "Invalid options to CGI script" error,
BR> but can't because of the typo in Mailman/Cgi/admindb.py, line
BR> 77:
BR> doc.AddItem(eader(2, "Invalid options to CGI script."))
BR> where Header is misspelled. That fixed the crash. But I
BR> think that this URL (admindb/) should work just like admindb
BR> without the slash -- it should say that you must specify a
BR> list.
Thanks for the bug report. It looks like Ken's fixed both problems for the next release.
-Barry

"BR" == Brian Ryner <bryner@uiuc.edu> writes:
BR> First of all, the bug I reported earlier happens if you access
BR> this URL:
BR> http://host/mailman/admindb/ <-- note trailing slash
BR> It tries to cause an "Invalid options to CGI script" error,
BR> but can't because of the typo in Mailman/Cgi/admindb.py, line
BR> 77:
BR> doc.AddItem(eader(2, "Invalid options to CGI script."))
BR> where Header is misspelled. That fixed the crash. But I
BR> think that this URL (admindb/) should work just like admindb
BR> without the slash -- it should say that you must specify a
BR> list.
Thanks for the bug report. It looks like Ken's fixed both problems for the next release.
-Barry
participants (2)
-
Barry A. Warsaw
-
Brian Ryner