Opened 8 years ago
#12461 new enhancement
Move svn_authz or rename to authz_policy
| Reported by: | Ryan J Ollos | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | version control | Version: | |
| Severity: | normal | Keywords: | permissions authzpolicy |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
While svn_fs and svn_prop were moved to tracopt.versioncontrol.svn, svn_authz remains in trac/versioncontrol. I think the justification was that svn_authz can be used for fine-grained access control in the repository browser for any type of repository. Note however that tracopt.perm.authz_policy can also be used for this purpose.
If the module is not svn-specific, we could consider renaming the modules to authz_policy and moving the options from the [svn] section to the [repositories] section. If the module should remain svn-specific, we should consider moving it to tracopt.versioncontrol.svn.
See also #12442.
I also plan to look at whether more code can be shared between tracopt.perm.authz_policy and trac.versioncontrol.svn_authz. I would like to add an AuthzFile class that can be used for reading and writing an AuthzFile, which I can use in th:TracSvnAuthzPlugin (which is important because it's a dependency of th:TracHacksPlugin, but much of the functionality of TracSvnAuthzPlugin is currently broken).
