Modify

Opened 3 years ago

Closed 3 years ago

#7881 closed defect (invalid)

RPC API via /rpc gives 'IndexError: pop from empty list'

Reported by: anonymous Owned by: osimons
Priority: normal Component: XmlRpcPlugin
Severity: minor Keywords: IndexError pop empty
Cc: olemis Trac Release: 0.12

Description

trac 0.12, XMLRPCPlugin 1.1.0, HTTPAuth 1.1, with/without patch 5445, Python 2.6

Clicking on the 'RPC API' link gives

Exception: ticket.update: pop from empty list Traceback (most recent call last): File "build\bdist.win32\egg\tracrpc\web_ui.py", line 101, in _dump_docs (method.signature, File "build\bdist.win32\egg\tracrpc\api.py", line 243, in _get_signature self._signature = '%s %s(%s)' % (RPC_TYPES[sig.pop()], self.name, ', '.join(args)) IndexError: pop from empty list

I'm logged in as admin, and the authenticated RPC user has XML_RPC permissions. I can access the API correctly via Java - it's just the API page that doesn't work. I haven't touched the underlying database.

The freaky thing is, this worked correctly 2 or 3 days ago. I don't think I changed anything with the exception of applying patch described in the above ticket.

I've since deleted all XMLRPC files and done a clean install, but the error still occurs.

I'm not fluent in Python, so I don't even know what to look for. Could someone give me a few pointers, please?

Attachments (0)

Change History (2)

comment:1 Changed 3 years ago by anonymous

Huh.

Looks like the line

yield (None, ((list, int, str), (list, int, str, dict), (list, int, str, dict, bool)), self.update)

needs to be replaced by

yield (None, ((list, int, str), (list, int, str, dict), (list, int, str, str, dict, bool)), self.update)

to account for the new author field, after applying the patch.

That was easy. Now to prove that black==white.

comment:2 Changed 3 years ago by osimons

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

Ah. An error from applying an incomplete patch from #5445. No point in keeping this ticket open - the deficiency should be marked on the original ticket instead.

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