Edgewall Software
Modify

Opened 13 years ago

Closed 12 years ago

Last modified 10 years ago

#3995 closed enhancement (fixed)

debug message for missing authz_file

Reported by: lh@… Owned by: Christian Boos
Priority: low Milestone: 0.10.5
Component: version control Version: 0.10
Severity: minor Keywords: authz_file permissions
Cc: Branch:
Release Notes:
API Changes:

Description

when the "authz_file" file configured in trac.ini is missing, or in my case misspelled I think a debug message saying "File not found" is in order.. rather than just:

DEBUG: Opening subversion file-system at /some/subversion/repository with scope /
WARNING: 403 Forbidden (Insufficient permissions to access /)

Attachments (0)

Change History (3)

comment:1 by hyuga <hyugaricdeau@…>, 12 years ago

Well, if the authz_file option is missing or blank, it should return a dummy Authorizer that just gives permission.

Furthermore, returning 403 Forbidden if the authz file can't be read is also the behavior in Subversion, though perhaps it would be a good idea to check if the file exists first and log an error (otherwise ConfigParser will just treat it like the file exists and is empty).

comment:2 by Christian Boos, 12 years ago

Status: newassigned

Fixed in r5244 (trunk).

comment:3 by Christian Boos, 12 years ago

Resolution: fixed
Status: assignedclosed

Ported to 0.l0-stable in r5246.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christian Boos.
The resolution will be deleted. Next status will be 'reopened'.
to as closed The owner will be changed from Christian Boos 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.