Launchpad fails when trying to authenticate with Launchpad plugin-enabled Bugzillas

Bug #270517 reported by Graham Binns
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Graham Binns

Bug Description

Since we implemented the UrlLib2Transport for XML-RPC, Launchpad will fail when trying to authenticate with Bugzilla. This has been masked by bug 269538 whenever we've tried to actually do the authentication.

The problem arises because, in BugzillaLPPlugin._authenticate() there is this line:

 auth_cookies = self._extractAuthCookie(self.xmlrpc_transport.last_response_headers['Set-Cookie'])

This works fine in tests, since our TestBugzillaXMLRPCTransport has a last_response_headers attribute, as did the xmlrpclib.Transport subclass that we used to use, which UrlLib2Transport replaced. UrlLib2Transport doesn't offer last_response_headers, so this line of code consistently fails.

We need to add a nice abstraction for retrieving cookies from UrlLib2Transport and also fix this line of code (and the related TestBugzillaXMLRPCTransport code).

Graham Binns (gmb)
description: updated
Graham Binns (gmb)
Changed in malone:
status: Triaged → In Progress
Revision history for this message
Graham Binns (gmb) wrote :

Landed in r7025.

Changed in malone:
status: In Progress → Fix Committed
Changed in malone:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.