Modify

Opened 7 years ago

Closed 5 years ago

#1987 closed enhancement (fixed)

enabling globbing user "*"

Reported by: trac@… Owned by: kisg
Priority: high Component: SvnAuthzAdminPlugin
Severity: trivial Keywords:
Cc: ThurnerRupert, diroussel, TimN Trac Release: 0.10

Description

Hi,

currently the pseudo-user "*" is missing in dropdown list. This catch-all is required if you want assign read-access to / to world.

eg

[project:/]
* = r

is currently not possible with this plugin :)

bye

Attachments (1)

svnauthzadminplugin-globbing-users.patch (971 bytes) - added by TimN 7 years ago.
Trivial patch to always add * to the member list as first element for paths, but not for groups

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by anonymous

  • Cc ThurnerRupert added

comment:2 Changed 7 years ago by ThurnerRupert

  • Priority changed from low to high

it is also necessary to restrict global from a subfolder:

[project:/my/very/secret]

  • =

comment:3 follow-up: Changed 7 years ago by anonymous

I have an existing authz file like this:

[groups]
my-commiters = rod,jane,freddy

[/]
* = r
@my-commiters = rw

But neither path shows up in the GUI. I guess it's the * that fails it?

comment:4 Changed 7 years ago by anonymous

  • Cc diroussel added

oop, that above comment was by me

comment:5 in reply to: ↑ 3 Changed 7 years ago by ThurnerRupert

for the plugin displays * on every position correctly. never tried a group with "-" though.

Replying to anonymous:

I have an existing authz file like this:

[groups]
my-commiters = rod,jane,freddy

[/]
* = r
@my-commiters = rw

But neither path shows up in the GUI. I guess it's the * that fails it?

Changed 7 years ago by TimN

Trivial patch to always add * to the member list as first element for paths, but not for groups

comment:6 Changed 7 years ago by TimN

  • Cc TimN added

The added patch allow for adding * as a user to a path.

Dashes in a group name are no problem, we use it here as well. Actually the setup you mention is quite similar to what we have here, so looks like a "should work"...

comment:7 Changed 5 years ago by ThurnerRupert

  • Resolution set to fixed
  • Status changed from new to closed

(In [5402]) close #1987, apply patch

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from kisg. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.