Opened 5 years ago

Closed 5 years ago

#30411 closed Cleanup/optimization (fixed)

Improve formatting of text tracebacks in technical 500 responses.

Reported by: Daniel Hahler Owned by: Jerrod Martin
Component: Error reporting Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

I think it would do better without extra newlines, and could just match the default output from Python itself.

Currently:

Traceback:

File "…/Vcs/django/django/template/base.py" in __init__
  663.                 token.Meh

During handling of the above exception ('str' object has no attribute 'Meh'), another exception occurred:

File "…/Vcs/django/django/core/handlers/exception.py" in inner
  34.             response = get_response(request)

File "…/Vcs/django/django/core/handlers/base.py" in _get_response
  145.                 response = self.process_exception_by_middleware(e, request)

File "…/Vcs/django/django/core/handlers/base.py" in _get_response
  143.                 response = response.render()

File "…/Vcs/django/django/template/response.py" in render
  106.             self.content = self.rendered_content

File "…/Vcs/django/django/template/response.py" in rendered_content
  83.         content = template.render(context, self._request)

File "…/Vcs/django/django/template/backends/django.py" in render
  61.             return self.template.render(context)

File "…/Vcs/django/django/template/base.py" in render
  171.                     return self._render(context)

File "…/Vcs/django/django/test/utils.py" in instrumented_test_render
  96.     return self.nodelist.render(context)

File "…/Vcs/django/django/template/base.py" in render
  940.                 bit = node.render_annotated(context)

File "…/Vcs/django/django/template/base.py" in render_annotated
  907.             return self.render(context)

File "…/Vcs/django/django/template/loader_tags.py" in render
  150.             return compiled_parent._render(context)

File "…/Vcs/django/django/test/utils.py" in instrumented_test_render
  96.     return self.nodelist.render(context)

File "…/Vcs/django/django/template/base.py" in render
  940.                 bit = node.render_annotated(context)

File "…/Vcs/django/django/template/base.py" in render_annotated
  907.             return self.render(context)

File "…/Vcs/django/django/template/loader_tags.py" in render
  127.         compiled_parent = self.get_parent(context)

File "…/Vcs/django/django/template/loader_tags.py" in get_parent
  124.         return self.find_template(parent, context)

File "…/Vcs/django/django/template/loader_tags.py" in find_template
  104.             template_name, skip=history,

File "…/Vcs/django/django/template/engine.py" in find_template
  125.                 template = loader.get_template(name, skip=skip)

File "…/Vcs/django/django/template/loaders/base.py" in get_template
  30.                     contents, origin, origin.template_name, self.engine,

File "…/Vcs/django/django/template/base.py" in __init__
  156.         self.nodelist = self.compile_nodelist()

File "…/Vcs/django/django/template/base.py" in compile_nodelist
  194.             return parser.parse()

File "…/Vcs/django/django/template/base.py" in parse
  449.                     raise self.error(token, e)

File "…/Vcs/django/django/template/base.py" in parse
  447.                     filter_expression = self.compile_filter(token.contents)

File "…/Vcs/django/django/template/base.py" in compile_filter
  563.         return FilterExpression(token, self)

File "…/Vcs/django/django/template/base.py" in __init__
  666.                                           "from '%s'" % (token[upto:], token))

Exception Type: TemplateSyntaxError at /a/login
Exception Value: Could not parse the remainder: '("foo")' from 'myecho("foo")'

Better:

Traceback (most recent call last):
  File "…/Vcs/django/django/template/base.py", line 663, in __init__
    token.Meh

During handling of the above exception ('str' object has no attribute 'Meh'), another exception occurred:
  File "…/Vcs/django/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "…/Vcs/django/django/core/handlers/base.py", line 145, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "…/Vcs/django/django/core/handlers/base.py", line 143, in _get_response
    response = response.render()
  File "…/Vcs/django/django/template/response.py", line 106, in render
    self.content = self.rendered_content
  File "…/Vcs/django/django/template/response.py", line 83, in rendered_content
    content = template.render(context, self._request)
  File "…/Vcs/django/django/template/backends/django.py", line 61, in render
    return self.template.render(context)
  File "…/Vcs/django/django/template/base.py", line 171, in render
    return self._render(context)
  File "…/Vcs/django/django/test/utils.py", line 96, in instrumented_test_render
    return self.nodelist.render(context)
  File "…/Vcs/django/django/template/base.py", line 940, in render
    bit = node.render_annotated(context)
  File "…/Vcs/django/django/template/base.py", line 907, in render_annotated
    return self.render(context)
  File "…/Vcs/django/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "…/Vcs/django/django/test/utils.py", line 96, in instrumented_test_render
    return self.nodelist.render(context)
  File "…/Vcs/django/django/template/base.py", line 940, in render
    bit = node.render_annotated(context)
  File "…/Vcs/django/django/template/base.py", line 907, in render_annotated
    return self.render(context)
  File "…/Vcs/django/django/template/loader_tags.py", line 127, in render
    compiled_parent = self.get_parent(context)
  File "…/Vcs/django/django/template/loader_tags.py", line 124, in get_parent
    return self.find_template(parent, context)
  File "…/Vcs/django/django/template/loader_tags.py", line 104, in find_template
    template_name, skip=history,
  File "…/Vcs/django/django/template/engine.py", line 125, in find_template
    template = loader.get_template(name, skip=skip)
  File "…/Vcs/django/django/template/loaders/base.py", line 30, in get_template
    contents, origin, origin.template_name, self.engine,
  File "…/Vcs/django/django/template/base.py", line 156, in __init__
    self.nodelist = self.compile_nodelist()
  File "…/Vcs/django/django/template/base.py", line 194, in compile_nodelist
    return parser.parse()
  File "…/Vcs/django/django/template/base.py", line 449, in parse
    raise self.error(token, e)
  File "…/Vcs/django/django/template/base.py", line 447, in parse
    filter_expression = self.compile_filter(token.contents)
  File "…/Vcs/django/django/template/base.py", line 563, in compile_filter
    return FilterExpression(token, self)
  File "…/Vcs/django/django/template/base.py", line 666, in __init__
    "from '%s'" % (token[upto:], token))

Exception Type: TemplateSyntaxError at /a/login
Exception Value: Could not parse the remainder: '("foo")' from 'myecho("foo")'

Change History (15)

comment:1 by Daniel Hahler, 5 years ago

There are two other issues here:

  1. I would like to have it more at the bottom
  2. the "Template error" part is not plain text:
    Template error:
    In template …/templates/base.html, error at line 10
       Could not parse the remainder: '("foo")' from 'myecho("foo")'
       1 : {# {% extends "base.html" %} #}
       2 :
       …
       12 : </html>
       13 :
    

comment:2 by Carlton Gibson, 5 years ago

Component: Core (Other)Error reporting
Easy pickings: set
Triage Stage: UnreviewedAccepted

OK, fine, improvements here welcome. Marking Easy Pickings... since it should just be formatting/template changes.

comment:3 by Naved Khan, 5 years ago

Owner: changed from nobody to Naved Khan
Status: newassigned

comment:4 by Daniel Hahler, 5 years ago

@Naved
Just to make sure it's not missed, there is an initial PR already at https://github.com/django/django/pull/11287.

comment:5 by Mariusz Felisiak, 5 years ago

Has patch: set
Owner: changed from Naved Khan to Daniel Hahler
Version: 2.2master

comment:6 by Mariusz Felisiak, 5 years ago

Patch needs improvement: set

comment:7 by Carlton Gibson, 5 years ago

Owner: Daniel Hahler removed
Status: assignednew

Deassigning due to inactivity. (PR should be a good starting point.)

Last edited 5 years ago by Carlton Gibson (previous) (diff)

comment:8 by 孙译喆, 5 years ago

I'm new to Django and I am intending to accept this issue. However I have a few questions to ask.
Firstly, how to make Django use technical_500.txt as a template? When I raise exceptions in views Django use technical_500.html instead.
Secondly, why is the existing pull request not accepted, and how can I improve it?

comment:9 by Jerrod Martin, 5 years ago

Owner: set to Jerrod Martin
Status: newassigned

comment:10 by Jerrod Martin, 5 years ago

New pull request with tests. https://github.com/django/django/pull/11596

comment:11 by Mariusz Felisiak, 5 years ago

Patch needs improvement: unset

comment:12 by Mariusz Felisiak, 5 years ago

Patch needs improvement: set

comment:13 by Mariusz Felisiak, 5 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

comment:14 by Mariusz Felisiak, 5 years ago

Summary: Improve traceback formatting in technical 500 text responsesImprove formatting of text tracebacks in technical 500 responses.

comment:15 by Mariusz Felisiak <felisiak.mariusz@…>, 5 years ago

Resolution: fixed
Status: assignedclosed

In c7bef16:

Fixed #30411 -- Improved formatting of text tracebacks in technical 500 templates.

Co-Authored-By: Daniel Hahler <git@…>

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