Modify

Opened 4 years ago

Closed 4 years ago

#7811 closed defect (fixed)

Failed to add download when no DOWNLOADS_ADMIN permission

Reported by: ak@… Owned by: Blackhex
Priority: normal Component: DownloadsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

When a user has DOWNLOADS_ADD but not DOWNLOADS_ADMIN, the upload fails with "Insufficient privileges to perform this operation". The problem is that download_created is called and check_permission is set to require DOWNLOADS_ADMIN for modification. The result is that the upload fails and even worse, the file is already uploaded but not probably "processed". When someone tries to upload a file, the plug-in reports that the directory for that ID already exists (well, actually the error message is a bit more cryptic). Apparently the ID is not probably assigned or something like that before the bug causes the exception...

Attachments (0)

Change History (2)

comment:1 Changed 4 years ago by ak@…

P.S.: Changing the check_permission in tags.py to the following line seems to fix this.

permissions = {'view' : 'TAGS_VIEW', 'modify' : 'TAGS_MODIFY'}

comment:2 Changed 4 years ago by Blackhex

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

Fixed in r9282.

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 Blackhex. 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.