Code review comment for lp:~mvo/software-center/cookie-jar

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Great work finding a solution so quickly mvo. I'm guessing you'll be able to add a related test for the trunk version.

08:18 < mvo> could someone do a s-c review for me? https://code.launchpad.net/~mvo/software-center/cookie-jar/+merge/112418 it unbreaks paypal so I would like to get it out of the door this morning
08:19 < noodles> mvo: did you reply to ricardo's comment on the bug?
08:19 * noodles checks
08:19 < mvo> I did
08:20 < noodles> mvo: but wouldn't there need to be a logout somewhere on USC itself? (or, how do people get to the pay page where the order summary is from within USC's embedded browser?)
08:21 < noodles> mvo: Ah, you're clearing the cookies on each start?
08:21 < noodles> Sweet.
08:21 < noodles> Approving.
08:22 < mvo> noodles: I don't know :) I mean, it might be that USC would have to offer this, but I don't know what it would have to do, go to a special page to logout? clear some special cookies? happy to work on this once I know what pay needs but that is probably trunk material
08:22 < mvo> thanks a lot noodles, the removal is a bit of a heavy handed approach, but its not a regression, before we had no cookies at all
08:22 < noodles> Yep.

review: Approve

« Back to merge proposal