Modify

Opened 14 years ago

Closed 13 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 Lang 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 14 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 13 years ago by osimons

Resolution: invalid
Status: newclosed

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.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain osimons.
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.