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

Revert "Limit twitter oauth scope" #4440

Merged
merged 1 commit into from
May 17, 2024

Conversation

tsmartt
Copy link
Contributor

@tsmartt tsmartt commented May 17, 2024

Closes https://github.com/brave-intl/creators-private-issues/issues/1822
Reverts #4421

This broke our X integration, we need at least these 2

@tsmartt tsmartt requested a review from jlbyrne May 17, 2024 20:43
@tsmartt tsmartt merged commit b58aeb2 into staging May 17, 2024
7 checks passed
@tsmartt tsmartt deleted the revert-4421-fix/limit-twitter-oauth-scope branch May 17, 2024 20:45
Copy link

The security team is monitoring all repositories for certain keywords. This PR includes the word(s) "oauth" and so security team members have been added as reviewers to take a look.

No need to request a full security review at this stage, the security team will take a look shortly and either clear the label or request more information/changes.

Notifications have already been sent, but if this is blocking your merge feel free to reach out directly to the security team on Slack so that we can expedite this check.

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

Successfully merging this pull request may close these issues.

4 participants