Modify ↓
#3995 closed enhancement (fixed)
debug message for missing authz_file
| Reported by: | 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: | |||
| Internal 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 , 18 years ago
comment:3 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Ported to 0.l0-stable in r5246.
Note:
See TracTickets
for help on using tickets.
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).