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