Edgewall Software
Modify

Opened 18 years ago

Closed 17 years ago

#4108 closed defect (wontfix)

System disclosure when calling /chrome/site/css

Reported by: ja@… Owned by: Jonas Borgström
Priority: low Milestone:
Component: general Version: 0.10
Severity: minor Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

Call /chrome/site/css will result in python trace error

Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/trac-0.10-py2.4.egg/trac/web/main.py", line 356, in dispatch_request
    dispatcher.dispatch(req)
  File "/usr/lib/python2.4/site-packages/trac-0.10-py2.4.egg/trac/web/main.py", line 224, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/lib/python2.4/site-packages/trac-0.10-py2.4.egg/trac/web/chrome.py", line 234, in process_request
    raise HTTPNotFound('File %s not found', filename)
HTTPException: 404 Not Found (File css not found)

Attachments (0)

Change History (6)

comment:1 by Emmanuel Blot, 18 years ago

Keywords: needinfo added

Not sure what you are expecting at this point:
chrome/site/css is usually a directory that contains site-specific CSS files.

Trac chrome handler does not list directory contents, it only serves files.

comment:2 by sid, 18 years ago

Priority: normallow
Severity: normalminor

Proposing to close as wontfix

comment:3 by Matthew Good, 18 years ago

Milestone: 0.10.3

I don't understand the summary's claim of "system disclosure", but I don't think that a traceback should be displayed for the 404 response, so we should look into why that's happening.

comment:4 by sid, 18 years ago

I believe he means that the Trac installation path is shown to the user in the traceback.

comment:5 by osimons, 17 years ago

Keywords: needinfo removed

This is no longer the case with 0.11:

  1. File not found will not give a traceback.
  2. If for some (other) reason a traceback occurs, it will only be available for users with TRAC_ADMIN permissions.

I did test it your example url on current 0.10.5dev, and the traceback appears to all users - the page only shows the traceback and nothing else, so this obviously does not get trapped by Trac in the proper manner.

Either someone feels this is important enough to be fixed for a possible 0.10.5 release, or we should close it as 'wontfix' and recommend to upgrade if this is a problem. Opinions?

comment:6 by Christian Boos, 17 years ago

Milestone: 0.10.5
Resolution: wontfix
Status: newclosed

I don't think this will change for 0.10.x.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.