[Patches] [ python-Patches-569574 ] plain text enhancement for cgitb

SourceForge.net noreply@sourceforge.net
Mon, 16 Jun 2003 11:00:12 -0700


Patches item #569574, was opened at 2002-06-16 00:46
Message generated for change (Comment added) made by hfastedge
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=569574&group_id=5470

Category: Library (Lib)
Group: Python 2.3
Status: Open
Resolution: None
Priority: 5
Submitted By: Skip Montanaro (montanaro)
Assigned to: Ka-Ping Yee (ping)
Summary: plain text enhancement for cgitb

Initial Comment:
Here's a patch to cgitb that allows you to enable plain
text output.  It adds an extra variable to the cgitb.enable
function and corresponding underlying functions.  To get
plain text invoke it as

    import cgitb
    cgitb.enable(format="text")

(actually, any value for format other than "html" will 
enable plain text output).  The default value is "html", so 
existing usage of cgitb should be unaffected.

I realize this isn't quite what you suggested, but it 
seemed to me worthwhile to keep such similar code 
together.

I'm not entirely certain I haven't fouled up the html
formatting.  It needs to be checked still.  Also still to come
is a doc change.

Skip


----------------------------------------------------------------------

Comment By: Hunter Peress (hfastedge)
Date: 2003-06-16 14:00

Message:
Logged In: YES 
user_id=479934

proper location to my bug
https://sourceforge.net/tracker/?func=detail&aid=654558&group_id=5470&atid=105470

----------------------------------------------------------------------

Comment By: Hunter Peress (hfastedge)
Date: 2003-06-16 13:58

Message:
Logged In: YES 
user_id=479934

http://mail.python.org/pipermail/python-dev/2003-June/036230.html

the need for this is clear and longstanding.

I filed a bug about this
https://sourceforge.net/tracker/?group_id=5470&atid=105470
please remove it as i never found this bug im writing on
before now

I even later wrote the patch for myself. im sure ping's
patch is better. 

I use this patch every day because it simply is flat out
better than python's current error mechanism.

----------------------------------------------------------------------

Comment By: Skip Montanaro (montanaro)
Date: 2002-08-02 13:45

Message:
Logged In: YES 
user_id=44345

I started messing around with this, but quickly figured out I didn't know 
what "main routine" belongs in traceback.  Moving lookup and scanvars 
there is no problem, but the rest of the functionality is almost entirely in the 
html() and text() routines.  Do you mean they should go in traceback?


----------------------------------------------------------------------

Comment By: Ka-Ping Yee (ping)
Date: 2002-07-09 23:33

Message:
Logged In: YES 
user_id=45338

I think enhanced text tracebacks would be great.

(I even have my own hacked-up one lying around
here somewhere -- it colourized the output.  I think
a part of me was waiting for an opportunity to
make enhanced tracebacks standard. The most
important enhancement IMHO is to show argument
values.)

I don't think the functionality belongs in cgitb,
though.  The main routine probably should go
in traceback; the common routines (scanvars
and lookup) can go there too.

----------------------------------------------------------------------

Comment By: Skip Montanaro (montanaro)
Date: 2002-07-09 19:48

Message:
Logged In: YES 
user_id=44345

Ping

How about you?  As the author I think you're in the best position to
decide on the merits of the patch...

Skip


----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2002-06-19 23:36

Message:
Logged In: YES 
user_id=6380

Unassigning -- I won't get to this before my vacation.

----------------------------------------------------------------------

Comment By: Skip Montanaro (montanaro)
Date: 2002-06-16 01:09

Message:
Logged In: YES 
user_id=44345

Okay, here's a correction to the first patch.  It fixes the logic
bug that corrupted the HTML output.  It also adds a little bit
of extra documentation.

Writing the documentation made me think that perhaps this
should be added to the traceback module as Guido
suggested with just a stub cgitb module that provides an 
enable function that calls the enable function in the 
traceback module with format="html".  The cgitb module
could then be deprecated.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=569574&group_id=5470