Modify

Opened 7 years ago

Closed 7 years ago

#7881 closed defect (invalid)

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

Reported by: anonymous Owned by: Odd Simon Simonsen
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 7 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 7 years ago by Odd Simon Simonsen

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 Odd Simon Simonsen.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.