Edgewall Software

Ticket #2490 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Python exception on malformed wiki URL

Reported by: molle.bestefich@… Owned by: cboos
Priority: low Milestone: 0.9.3
Component: wiki system Version: 0.9.2
Severity: trivial Keywords:
Cc:

Description

Go to this URL:

http://projects.edgewall.com/trac/attachment/wiki/TracMercurial/

And you'll get this:

Traceback (most recent call last):
  File "/usr/lib/python2.3/site-packages/trac/web/modpython_frontend.py", line 206, in handler
    dispatch_request(mpr.path_info, mpr, env)
  File "/usr/lib/python2.3/site-packages/trac/web/main.py", line 139, in dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.3/site-packages/trac/web/main.py", line 107, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/lib/python2.3/site-packages/trac/attachment.py", line 273, in process_request
    self._render_view(req, attachment)
  File "/usr/lib/python2.3/site-packages/trac/attachment.py", line 390, in _render_view
    req.check_modified(attachment.time)
  File "/usr/lib/python2.3/site-packages/trac/web/api.py", line 103, in check_modified
    etag = 'W"%s/%d/%s"' % (self.authname, timesecs, extra)
TypeError: int argument required

Where you really would have expected:

An HTTP 404 response and a fancy 'not found' page for the user to look at :-).

Attachments

Change History

Changed 3 years ago by cboos

  • owner changed from jonas to cboos
  • status changed from new to assigned
  • milestone set to 0.9.3

Fortunately this has nothing to do with TracMercurial :) You get this for any attachment/*/*/ URL...

Let's see... ok. I fixed it, but have no connectivity to svn.edgewall.com right now...

Changed 3 years ago by cboos

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in r2656 and r2659.

Add/Change #2490 (Python exception on malformed wiki URL)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.