python: no ampersand in plaintext signature, no escaping in Authentication header about oauth

[GoogleCodeExporter]

Verified, patch fixes plaintext signature method. I was facing this issue as 
well.

Original comment by [email protected] on 24 Jun 2008 at 10:11

[GoogleCodeExporter]

I confirm that bug, here is a more complete patch which handle PLAINTEXT 
signature in a consistent way and 
solve a bug with a mutable argument in OAuthRequest.from_request()

Someone can commit those changes? Otherwise I can do this (tests need to be 
fixed too, they seems broken for 
now). Thanks.

Original comment by [email protected] on 30 Jun 2008 at 11:34

Attachments:

• patch_oauth.20080701.diff

[GoogleCodeExporter]

Patch plaintext.signature.ampersand.diff looks great and I've checked it in:

http://code.google.com/p/oauth/source/detail?r=588

Thanks atregoubenko!

Original comment by leah.culver on 8 Jul 2008 at 4:32

• Changed state: Accepted

[GoogleCodeExporter]

larlet - I've added one of your changes, the check for the OAuthError 
specifically.
Can you provide more reasoning for your other two changes? I'm closing this 
ticket,
but feel free to open another for those issues.

Thanks!

Original comment by leah.culver on 8 Jul 2008 at 4:37

• Changed state: Fixed

[GoogleCodeExporter]

Original comment by leah.culver on 14 Jan 2009 at 8:33

• Added labels: Lib-Python