durusmail: quixote-users: cgitb patch
Misc feedback
2002-01-28
2002-01-28
2002-01-29
David Ascher (2 parts)
cgitb patch
2002-01-30
2002-01-30
2002-01-30
2002-01-30
2002-01-30
2002-01-30
2002-01-30
cgitb patch
Greg Ward
2002-01-30
On 30 January 2002, Andrew Kuchling said:
>  def format_internal_error (self, request, error_message):
>       return error_message
>
> And finish_failed_request would then contain:
>
>        if not self.config.display_exceptions:
>             # Use most-secure, internal error message
>        elif self.config.display_exceptions == 'plain':
>             # existing plain text version
>        elif self.config.display_exceptions == 'html'
>             # use cgitb

Yup, looks good to me.

> Part 3 of the patch would be changing the config.Config class to check
> that display_exceptions is one of "plain", "html", or .

Or 1 (== "plain"), for backward compatibility.  We have *users* now, we
have to start worrying about that...

        Greg
--
Greg Ward - software developer                gward@mems-exchange.org
MEMS Exchange                            http://www.mems-exchange.org


reply