Edgewall Software
Modify

Opened 19 years ago

Closed 19 years ago

Last modified 18 years ago

#2223 closed defect (fixed)

trac-admin bypasses PermissionStore

Reported by: Emmanuel Blot Owned by: Jonas Borgström
Priority: normal Milestone: 0.9
Component: general Version: 0.9b2
Severity: major Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

trac-admin as per [2353] bypasses the new PermissionStore component to retrieve, add and revoke permissions. It therefore ignores any custom PermissionStore extension: Trac engine uses the extension, but trac-admin keeps dealing with the SQL backend.

I attach a quite ugly patch to this ticket, as a workaround. I'm sure a better, more "Python" implementation can be written. This patch can be used to enable the TracLdap extension, till a cleaner/better implementation is written by Trac developers.

I think the target should be 0.9, as this issue prevents from deploying a permission component.

Attachments (1)

admin.2.patch (3.2 KB ) - added by Emmanuel Blot 19 years ago.
Patch for trac/scripts/admin.py, to use PermissionStore components

Download all attachments as: .zip

Change History (4)

by Emmanuel Blot, 19 years ago

Attachment: admin.2.patch added

Patch for trac/scripts/admin.py, to use PermissionStore components

comment:1 by Emmanuel Blot, 19 years ago

Sorry, the first patchfile is invalid. Cannot delete it

comment:2 by Jonas Borgström, 19 years ago

Status: newassigned

comment:3 by Jonas Borgström, 19 years ago

Resolution: fixed
Status: assignedclosed

Thanks, applied in [2358].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jonas Borgström.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jonas Borgström 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.