Patch: Use same instance of OAuthProvider in OAuth dance.

ElderB / google-refine

Automatically exported from code.google.com/p/google-refine

0 stars 0 forks source link

Patch: Use same instance of OAuthProvider in OAuth dance. #614

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Currently a new instance of oauth.signpost.OAuthProvider is created during 
every step of the OAuth dance. This breaks for OAuth 1.0a since it stores 
state. This patch causes a single instance of oauth.signpost.OAuthProvider to 
be used for each com.google.refine.oauth.Provider instance.

Original issue reported on code.google.com by s...@google.com on 19 Sep 2012 at 6:25

Attachments:

patch

GoogleCodeExporter commented 9 years ago

Original comment by tfmorris on 19 Sep 2012 at 11:11

Changed state: Accepted
Added labels: Type-Patch, Milestone-2.6
Removed labels: Type-Defect

GoogleCodeExporter commented 9 years ago

Original comment by tfmorris on 19 Sep 2012 at 11:17

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Thanks for the patch.  Patch applied in r2566.

Original comment by tfmorris on 19 Sep 2012 at 11:18