Edgewall Software
Modify

Opened 16 years ago

Closed 13 years ago

Last modified 11 years ago

#8269 closed enhancement (worksforme)

"authz_module_name" should be more verbosely documented

Reported by: sebastian@… Owned by:
Priority: normal Milestone:
Component: version control Version: 0.11.4
Severity: normal Keywords: svnauthz, documentation, authzsourcepolicy
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

I just completed a two hour hunt for a bug which, in the end, wasn't one. The problem was that I could anonymously access my SVN repository but Trac's source browser gave me just an uninformative error message: "Error: Forbidden". (see screenshot; "anonymous" has all the necessary rights, btw. Especially "BROWSER_VIEW")

It turned out that I forgot to set the configuration option authz_module_name which caused the problem.

It tried to browse a repository called "ixml" and had the following "authz" file.

[/]
manski=rw

[ixml:/]
*=r
manski=rw

...

So I have two suggestions:

  • Enhance the error output displayed to the user (see screenshot) or at least log the error. The error message of the PermissionError is completely lost.
  • Add some more documentation to authz_module_name stating at least the "module" means "repository name" (like specified here) and that this value can't be automatically detected. A IMHO better way would be to fixe/implement ticket #4997.

BTW: I'm using Python 2.6.

Attachments (1)

error-forbidden.png (7.9 KB ) - added by sebastian@… 16 years ago.
Screenshot of the error message

Download all attachments as: .zip

Change History (6)

by sebastian@…, 16 years ago

Attachment: error-forbidden.png added

Screenshot of the error message

comment:1 by Christian Boos, 15 years ago

Keywords: svnauthz added
Milestone: 0.11.6

comment:2 by Christian Boos, 14 years ago

Keywords: documentation added
Milestone: next-minor-0.12.xunscheduled

comment:3 by Christian Boos, 13 years ago

Resolution: worksforme
Status: newclosed

I re-read the documentation for authz_module_name which states:

The module prefix used in the authz_file for the default repository. If left empty, the global section is used.

Well, that's clear enough for me. Please reopen if you have a better phrasing.

As for the missing permission error message, I couldn't reproduce it.

comment:4 by Christian Boos, 13 years ago

Milestone: unscheduled

comment:5 by Ryan J Ollos, 11 years ago

Keywords: authzsourcepolicy added

Modify Ticket

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