It strikes me that my description above could use some fleshing out.  Here goes.

The bug was triggered when the OpenID provider chooses to present the user with 
a form with a single Continue button to click on instead of redirecting the 
user directly to Launchpad (because the redirect would have contained a 
Location header that is too long for some browsers).

Launchpad wasn't expecting this behavior and couldn't handle it.  The solution 
was to gather the data needed to complete the OpenID handshaking from POST 
requests as well as GET requests (the data ends up in different data structures 
in the two scenarios).

There was also a bug in the development OpenID provider such that it did not 
set the content-type correctly when generating the aforementioned 
continue-button-containing-form, so the user only saw raw HTML.
-- 
https://code.launchpad.net/~benji/launchpad/bug-597324/+merge/30330
Your team Launchpad code reviewers is requested to review the proposed merge of 
lp:~benji/launchpad/bug-597324 into lp:launchpad/devel.

_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help   : https://help.launchpad.net/ListHelp

Reply via email to