Skip to content

Conversation

@qnub
Copy link

@qnub qnub commented Dec 16, 2016

Got error request() got an unexpected keyword argument 'oauth_callback' when try to call .fetch_request_token(url=self.request_token_url, oauth_callback=self.redirect_uri)

Got error `request() got an unexpected keyword argument 'oauth_callback'` when try to call `.fetch_request_token(url=self.request_token_url, oauth_callback=self.redirect_uri)`
@coveralls
Copy link

coveralls commented Dec 16, 2016

Coverage Status

Coverage remained the same at 86.758% when pulling 08cfa2f on qnub:patch-1 into 476ba75 on requests:master.

@Lukasa
Copy link
Member

Lukasa commented Dec 16, 2016

This change is definitely not right. oauth_callback is not a parameter we accept anywhere. What are you trying to achieve?

@Lukasa Lukasa closed this Dec 16, 2016
@qnub
Copy link
Author

qnub commented Dec 17, 2016

@Lukasa sad, because is only way to force twitter to direct user back to URL other than set on apps.twitter.com.

Check «Step 1» here https://dev.twitter.com/web/sign-in/implementing:

The only unique parameter in this request is oauth_callback, which must be a URL-encoded version of the URL you wish your user to be redirected to when they complete step 2.

@Lukasa
Copy link
Member

Lukasa commented Dec 17, 2016

If you want to set a parameter, you should use params= instead. =)

@qnub
Copy link
Author

qnub commented Dec 23, 2016

Have you tryed to do that? Seems it's not work.

@Lukasa
Copy link
Member

Lukasa commented Dec 23, 2016

When you say it doesn't work, what specifically do you mean? Additionally, have you tried setting data= instead of params=?

@qnub qnub deleted the patch-1 branch December 23, 2016 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants