Opened 7 years ago

Closed 6 years ago

#7529 closed (fixed)

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

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

Description

It would be nice if it showed them.

Attachments (3)

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

Download all attachments as: .zip

Change History (16)

Changed 7 years ago by Alex

Now it shows them just like POST, etc..

comment:1 Changed 7 years ago by Alex

  • Has patch set
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 7 years ago by anonymous

  • Component changed from Uncategorized to Metasystem
  • Triage Stage changed from Unreviewed to Design decision needed

comment:3 Changed 7 years ago by Bastian Kleineidam <calvin@…>

  • 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 Changed 7 years ago by mtredinnick

  • Triage Stage changed from Design decision needed to Accepted

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 Changed 7 years ago by Alex

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

comment:6 Changed 7 years ago by mir

  • Keywords 1.0-unsure added

comment:7 Changed 7 years ago by Alex

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

comment:9 Changed 7 years ago by julianb

  • milestone set to 1.0 maybe

comment:10 Changed 7 years ago by jacob

  • Component changed from Metasystem to Core framework
  • Keywords 1.0-unsure removed
  • milestone changed from 1.0 maybe to post-1.0

comment:11 Changed 6 years ago by Alex

  • Owner changed from nobody to Alex

comment:12 Changed 6 years ago by anonymous

  • milestone post-1.0 deleted

Milestone post-1.0 deleted

Changed 6 years ago by Alex

Changed 6 years ago by Alex

comment:13 Changed 6 years ago by Alex

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r10271

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