Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Last release is broken #139

Closed
Diegslapasteque opened this issue Aug 11, 2021 · 2 comments
Closed

Last release is broken #139

Diegslapasteque opened this issue Aug 11, 2021 · 2 comments

Comments

@Diegslapasteque
Copy link

Diegslapasteque commented Aug 11, 2021

Hey,
I made a pull request last week about Twitter, but I don't test it because I thought the Twitter docs were clear enough.
Sadly, it was not.
So now, when we make an oauth connection with Twitter, we get this :

{
    "errors":[{
        "code":32,
        "message":"Could not authenticate you."
    }]
}".

This error is documented here : https://developer.twitter.com/en/support/twitter-api/error-troubleshooting (search for "code 32" in the page).
I haven't already find a solution to fix that, but I will made a PR a soon as possible.

Sorry for that.

@bitmanid
Copy link

I got same problem. After a few hour debugging and searching in commit history, the problem come from file commit on server at temporaryCredentialsProtocolHeader. It was commit from 10 days ago

8bf4d9a#diff-fb33b1b907a4b627dbcc1965898b866ab2be2ac4d4bb611bd47caf39137b5d08

and here is the code:

$parameters = array_merge(
      $this->baseProtocolParameters(),
      $this->additionalTemporaryCredentialsProtocolParameters(),
      [
          'oauth_callback' => $this->clientCredentials->getCallbackUri(),
      ]
  );

i think it need twitter application scope

'scope' => 'your-application-scope' // optional ('read', 'write'), defaults to 'read'

but in my case i'm using the previous code, i haven't try to add application scope.

@bencorlett
Copy link
Member

bencorlett commented Aug 12, 2021 via email

Diegslapasteque pushed a commit to Diegslapasteque/oauth1-client that referenced this issue Aug 12, 2021
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

No branches or pull requests

3 participants