#12309 closed defect (fixed)
Permissions are reset everytime the plugin UI is opened
Reported by: | Owned by: | Ryan J Ollos | |
---|---|---|---|
Priority: | high | Component: | SvnAuthzAdminPlugin |
Severity: | critical | Keywords: | |
Cc: | Trac Release: | 1.0 |
Description
I am setting a permission in authz either using the plugin UI or by manually editing the file.
[groups] srvadmin = peter, pan [services:/] @srvadmin = r john = rw jane = rw
After that, if i open/launch "Subversion Access" UI; the permissions for the group or user is set to blank.
So it shows up as:
[groups] srvadmin = peter, pan [services:/] @srvadmin = john = jane =
Can it be fixed?
Source used: https://trac-hacks.org/changeset/14583/svnauthzadminplugin/1.0/svnauthz
Attachments (0)
Change History (8)
comment:1 Changed 10 years ago by
comment:2 Changed 10 years ago by
I spent a few hours on this plugin last weekend and decided it was in such bad shape that it pretty much needs an entire rewrite. That's fine with me anyway because I don't like the interface and it's GPL licensed. I prefer to work on BSD license because I can freely migrate code to the Trac core.
I'll try to patch this issue over the weekend, but overall I'm not optimistic about the plugin.
comment:3 Changed 10 years ago by
Hi,
If you think this plugin will not work.
Can i ask you to suggest an alternative for SVN Authz file UI?
comment:4 Changed 10 years ago by
If you just want to edit the Authz file through the web interface you could try TracSvnAuthzPlugin. It appears to be similar to FineGrainedPageAuthzEditorPlugin, which might work as well for the purpose of editing the Authz file, or could at least be hacked to make it work.
comment:5 Changed 10 years ago by
This is a regression bug introduced in diff:svnauthzadminplugin/1.0/svnauthz/io.py@14491:14494
In _parse_path_acl
the variables acl
and aclstr
are merged into acl
, resulting in one overwriting the other.
source:svnauthzadminplugin/1.0/svnauthz/io.py@14494:121-122#L120
comment:6 Changed 10 years ago by
Thanks for tracking that down. I was being careless in trying to cleanup the codebase. I'll commit the fix today.
comment:8 Changed 10 years ago by
Please open a new ticket if you spot any other issues with the plugin. We'd like to use the plugin on trac-hacks.org, so eventually I'll spend the time to redesign the plugin and add unit tests.
Any updates, dude?