Ticket #4722 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

post_process_request not preserve content_type, which cause text/plain templates fail to render.

Reported by: jiangxin Assigned to: jiangxin
Priority: highest Component: TracCoSignPlugin
Severity: blocker Keywords:
Cc: Trac Release: 0.11

Description

post_process_request in traccosign/accountldap.py carelessly set content_type to None. It's bad if template is not a html template but a text template.

This bug cause plain text output of source code revision log raise a exception:

Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/trac/web/main.py", line 459, in _dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.5/site-packages/trac/web/main.py", line 237, in dispatch
    content_type)
  File "/usr/lib/python2.5/site-packages/trac/web/chrome.py", line 740, in render_template
    template = self.load_template(filename, method=method)
  File "/usr/lib/python2.5/site-packages/trac/web/chrome.py", line 724, in load_template
    return self.templates.load(filename, cls=cls)
  File "/var/lib/python-support/python2.5/genshi/template/loader.py", line 227, in load
    filename, encoding=encoding)
  File "/var/lib/python-support/python2.5/genshi/template/loader.py", line 265, in _instantiate
    allow_exec=self.allow_exec)
  File "/var/lib/python-support/python2.5/genshi/template/markup.py", line 67, in __init__
    allow_exec=allow_exec)
  File "/var/lib/python-support/python2.5/genshi/template/base.py", line 406, in __init__
    raise TemplateSyntaxError(e.msg, self.filepath, e.lineno, e.offset)
TemplateSyntaxError: not well-formed (invalid token): line 1, column 1 (/usr/lib/python2.5/site-packages/trac/versioncontrol/templates/revisionlog.txt, line 1)

Attachments

Change History

02/28/09 15:45:13 changed by jiangxin

  • status changed from new to closed.
  • resolution set to fixed.

(In [5325]) Fixed #4722: wrong content_type return value from post_process_request.

02/28/09 15:51:46 changed by jiangxin

(In [5326]) Upgrade to 0.1.2-1.5325. See #4722.


Add/Change #4722 (post_process_request not preserve content_type, which cause text/plain templates fail to render.)




Change Properties
Action