Fix for missing oauth_timestamp parameter #375

Merged
merged 1 commit into from Nov 21, 2012

Projects

None yet

2 participants

@jplock
Contributor
jplock commented Nov 20, 2012

If one of the request parameters is called "timestamp", the "oauth_timestamp" OAuth parameter will get removed during the parameter cleanup loop.

@jplock jplock If one of the request parameters is called "timestamp", the "oauth_ti…
…mestamp" OAuth parameter will get removed during the parameter cleanup loop.
8c3e9cb
@mikeal mikeal merged commit e6c7aa5 into request:master Nov 21, 2012
@jplock
Contributor
jplock commented Nov 21, 2012

Technically this could happen on any of the oauth_* variables. This fixes my immediate need, so thanks for merging it in, but the whole variable population part should probably be refactored at some point in my opinion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment