Opened 9 years ago

Closed 9 years ago

#12300 closed defect (fixed)

Internal Error while clicking on Plugin UI

Reported by: anonymous Owned by: Ryan J Ollos
Priority: high Component: SvnAuthzAdminPlugin
Severity: blocker Keywords:
Cc: Trac Release: 1.0


After Installing the plugin with Trac 1.0.5 i get the error:

Found this in trac.log:

2015-05-05 07:54:00,916 Trac[main] ERROR: Internal Server Error:
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/trac/web/", line 513, in _dispatch_request
  File "/usr/lib/python2.6/site-packages/trac/web/", line 222, in dispatch
    resp = chosen_handler.process_request(req)
  File "/usr/lib/python2.6/site-packages/trac/admin/", line 120, in process_request
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 70, in render_admin_panel
    self.authz = self._get_authz(req)
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 389, in _get_authz
    return AuthzFileReader().read(self.authz_file)
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 19, in read
    return parser.parse()
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 50, in parse
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 78, in _parse_root
    self._parse_path_acl(m, line)
  File "/usr/lib/python2.6/site-packages/svnauthz/", line 121, in _parse_path_acl
    subject, acl = line.split('=')
ValueError: too many values to unpack

Attachments (0)

Change History (5)

comment:1 Changed 9 years ago by Ryan J Ollos

Status: newaccepted

This plugin needs a lot of work. I doubt it is useful even after fixing this one issue, but I will try to fix it by this weekend.

comment:2 Changed 9 years ago by Ryan J Ollos

Could you please post your authz file so that I can try to reproduce?

comment:3 Changed 9 years ago by vidr@…

Sure, heres it. some details masked...

adadmin = CN=Admin,OU=Standard,OU=Groups,DC=example,DC=com

srvadmin = @adadmin

@srvadmin = r
john = rw
jane = rw
Last edited 9 years ago by Ryan J Ollos (previous) (diff)

comment:4 Changed 9 years ago by Ryan J Ollos

In 14582:

1.0dev: Don't split option values containing =.

This avoids a "too many values to unpack" exception when the option value contains a = character.
Note that the plugin still doesn't understand the [aliases] section. Refs #12300.

comment:5 Changed 9 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Note: See TracTickets for help on using tickets.