Python: Oauth request Authorization header contains non-oauth prefixed params

[longxiang]

Python: Oauth request Authorization header contains non-oauth prefixed params

[GoogleCodeExporter]

Original comment by andyster on 15 Jul 2008 at 11:17

• Added labels: Lib-Python
• Removed labels: ****

[GoogleCodeExporter]

I ran into the same issue.

Original comment by [email protected] on 5 Sep 2008 at 8:35

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

I too ran into the same issue.. so had to find a way around... I had to use

del oauth_request.parameters[param] # url query param to be removed

before calling oauth_request.to_header()

Original comment by [email protected] on 30 Oct 2008 at 8:01

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

I submitted a patch almost 4 months ago -- any status on whether this minute 
bug will be fixed or anything like 
that? Looks like others have faced the same issue and might be wasting time 
figuring out what the cause of this 
bug is.

Original comment by [email protected] on 30 Oct 2008 at 8:17

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

I ran into the same issue, and ended up coding almost the same patch (although a
little bit uglier, like the php version). Any change of being fixed?

Original comment by [email protected] on 4 Nov 2008 at 2:12

• Added labels: ****
• Removed labels: ****

Attachments:

• patch.diff

[GoogleCodeExporter]

additionally, in order to use yahoo's yql webservice, I also had to modify the
"to_url()" method. I'm not sure if this change breaks anything else..

Cumulative patch attached.

Original comment by [email protected] on 1 Dec 2008 at 4:18

• Added labels: ****
• Removed labels: ****

Attachments:

• patch2.diff

[GoogleCodeExporter]

Original comment by leah.culver on 9 Jan 2009 at 12:30

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Thanks! This has been fixed. I'm not sure if the auth header should contain any
non-oauth params, so I hope nobody is expecting it to.

I'm not going to change the to_url() method. If this is a big problem, please 
file a
new ticket... this should probably be only oauth params - not the other way 
around.

Thanks all.

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

• Changed state: Fixed
• Added labels: ****
• Removed labels: ****