Edgewall Software

Ticket #7391 (new defect)

Opened 5 months ago

Last modified 3 weeks ago

renamed plugin disable commands in trac.ini [components] silently fail, a security issue

Reported by: anonymous Owned by: anonymous
Priority: low Milestone: 0.11.3
Component: general Version: 0.11
Severity: trivial Keywords: security
Cc:

Description

On upgrading to trac 0.11, I found I had to rename:

   webadmin.plugin.pluginadminpage=disabled

to

trac.admin.web_ui.PluginAdminPanel=disabled

The problem here is that I found this by noticing that trac 0.11 was allowing uploads.

There was no complaint about the old disabled line not being relevant any more, and no upgrade documentation to warn that if we locked things down in webadmin we now need to rename the lines in the config file.

I think the lack of warning (in code or in documentation) is a security risk to people upgrading.

Attachments

Change History

Changed 5 months ago by anonymous

  • owner set to anonymous
  • status changed from new to assigned

ccc

Changed 5 months ago by Piotr Kuczynski <piotr.kuczynski@…>

  • keywords security added
  • version set to 0.11
  • component changed from general to admin/web
  • severity changed from normal to critical
  • milestone set to 0.11.1

Changed 4 weeks ago by cboos

  • priority changed from normal to low
  • severity changed from critical to major
  • milestone changed from 0.11.2 to 0.11.3

Well, hm, I think it's a bit late to bother with upgrades from WebAdmin, but if someone contributes a patch, why not.

Changed 4 weeks ago by rblank

I have been tempted several times to just close this as wontfix. It's the site admin's job to check the site thoroughly after an upgrade, after all. At most, add a warning to the upgrade instructions for 0.11.

Changed 3 weeks ago by anonymous

  • status changed from assigned to new
  • component changed from admin/web to general
  • severity changed from major to trivial

Add/Change #7391 (renamed plugin disable commands in trac.ini [components] silently fail, a security issue)

Author



Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will change from anonymous. Next status will be 'new'
Next status will be 'assigned'
 
Note: See TracTickets for help on using tickets.