Opened 16 years ago

Closed 16 years ago

#7529 closed (fixed)

Debug 500 Page Shows POST, GET, and COOKIES but not FILES

Reported by: Alex Gaynor Owned by: Alex Gaynor
Component: Core (Other) Version: dev
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

It would be nice if it showed them.

Attachments (3)

files.diff (901 bytes ) - added by Alex Gaynor 16 years ago.
Now it shows them just like POST, etc..
debug-files.diff (2.6 KB ) - added by Alex Gaynor 16 years ago.
debug-files.2.diff (3.5 KB ) - added by Alex Gaynor 16 years ago.

Download all attachments as: .zip

Change History (16)

by Alex Gaynor, 16 years ago

Attachment: files.diff added

Now it shows them just like POST, etc..

comment:1 by Alex Gaynor, 16 years ago

Has patch: set

comment:2 by anonymous, 16 years ago

Component: UncategorizedMetasystem
Triage Stage: UnreviewedDesign decision needed

comment:3 by Bastian Kleineidam <calvin@…>, 16 years ago

Patch needs improvement: set

The patch prints out full file contents in the debug page. This will be unpleasant with big files.

I suggest limiting the amount of printed file data to the first 200 bytes or so, and to replace binary characters with hexadecimal values.

comment:4 by Malcolm Tredinnick, 16 years ago

Triage Stage: Design decision neededAccepted

Files are huge amounts of data. A patch to show the number of uploaded files would be reasonable, but we shouldn't be showing their contents (even the first 200 bytes is really pretty useless most of the time). It's just too much opaque information. If somebody wants more information, they can do their own 500 handling without a lot of trouble (or put debugging prints into their code based on the exception traceback).

comment:5 by Alex Gaynor, 16 years ago

To my understanding the unicode method on the file objects should be printing out the filename, which is acceptable to print out.

comment:6 by Michael Radziej, 16 years ago

Keywords: 1.0-unsure added

comment:7 by Alex Gaynor, 16 years ago

Small note: I have tested this post-2070, so it does indeed print out the file names.

comment:9 by Julian Bez, 16 years ago

milestone: 1.0 maybe

comment:10 by Jacob, 16 years ago

Component: MetasystemCore framework
Keywords: 1.0-unsure removed
milestone: 1.0 maybepost-1.0

comment:11 by Alex Gaynor, 16 years ago

Owner: changed from nobody to Alex Gaynor

comment:12 by (none), 16 years ago

milestone: post-1.0

Milestone post-1.0 deleted

by Alex Gaynor, 16 years ago

Attachment: debug-files.diff added

by Alex Gaynor, 16 years ago

Attachment: debug-files.2.diff added

comment:13 by Alex Gaynor, 16 years ago

Resolution: fixed
Status: newclosed

Fixed in r10271

Note: See TracTickets for help on using tickets.
Back to Top