Edgewall Software

Ticket #3995 (closed enhancement: fixed)

Opened 2 years ago

Last modified 19 months ago

debug message for missing authz_file

Reported by: lh@… Owned by: cboos
Priority: low Milestone: 0.10.5
Component: version control Version: 0.10
Severity: minor Keywords: authz_file permissions
Cc:

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

Change History

Changed 20 months ago by hyuga <hyugaricdeau@…>

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).

Changed 19 months ago by cboos

  • status changed from new to assigned

Fixed in r5244 (trunk).

Changed 19 months ago by cboos

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

Ported to 0.l0-stable in r5246.

Add/Change #3995 (debug message for missing authz_file)

Author



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