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

OAuth test fails due to Twitter API issue #7470

Closed
5 of 6 tasks
mtrezza opened this issue Jul 23, 2021 · 2 comments · Fixed by #7472
Closed
5 of 6 tasks

OAuth test fails due to Twitter API issue #7470

mtrezza opened this issue Jul 23, 2021 · 2 comments · Fixed by #7472
Labels
state:released Released as stable version state:released-beta Released as beta version type:meta Non-code issue

Comments

@mtrezza
Copy link
Member

mtrezza commented Jul 23, 2021

New Issue Checklist

Issue Description

Currently, the OAuth test fails due to an issue with the Twitter API endpoint that is used for testing a failed OAuth response.
The response is an incorrectly formatted JSON string, so parsing fails.

This happened at least once some time ago, so the endpoint to test against has been changed. If this keeps happening with the Twitter API, we may change to an OAuth API endpoint of another provider for testing.

Steps to reproduce

Run test

GET https://api.twitter.com/1.1/account/settings.json

Actual Outcome

{"errors":"{\"message\":\"Could not authenticate you\",\"code\":32}"}

Expected Outcome

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

Failing Test Case / Pull Request

  • 🤩 I submitted a PR with a fix and a test case.
  • 🧐 I submitted a PR with a failing test case.

Environment

Server

  • Parse Server version: master 1594afe
  • Operating system: n/a
  • Local or remote host (AWS, Azure, Google Cloud, Heroku, Digital Ocean, etc): n/a

Database

  • System (MongoDB or Postgres): n/a
  • Database version: n/a
  • Local or remote host (MongoDB Atlas, mLab, AWS, Azure, Google Cloud, etc): n/a

Client

  • SDK (iOS, Android, JavaScript, PHP, Unity, etc): n/a
  • SDK version: n/a

Logs

(none)

@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 5.0.0-beta.1

@parseplatformorg parseplatformorg added the state:released-beta Released as beta version label Nov 1, 2021
@parseplatformorg
Copy link
Contributor

🎉 This change has been released in version 5.0.0

@parseplatformorg parseplatformorg added the state:released Released as stable version label Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state:released Released as stable version state:released-beta Released as beta version type:meta Non-code issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants