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

Fix authorization for searchTweets/Add non recursive options for searchTweets and getUserTimeline #459

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

hemisphire
Copy link

  1. Fix authorization for searchTweets

All searches now require OAuth2.0

  1. Add non recursive options for searchTweets and getUserTimeline

We have a use case where the user decides how many results they want, as a popular query can return more than the 10,000 monthly limit in Basic. Since the pagination token is available from TweetList.getMeta().getNextToken(), we can call additionalParameters.setPaginationToken(paginationToken).

2. Add non recursive options for searchTweets and getUserTimeline
@hemisphire
Copy link
Author

And FYI, the only way I can currently get authorization working is to set

  1. apiKey
  2. apiSecretKey
  3. bearerToken

@RedouaneBALI
Copy link

Hello @hemisphire I'm sorry I just realized that I lost the access to my GitHub account @redouane59 because I changed my laptop + my smartphone last month. It is a mess because the Authenticator App code of my new phone is not accepted by GitHub because it was not registered before. I don't know if I will be able to recover it to validate the PR...

@RedouaneBALI
Copy link

https://docs.github.com/en/authentication/securing-your-account-with-two-factor-authentication-2fa/recovering-your-account-if-you-lose-your-2fa-credentials

Already checked it but that's not enough. I opened a ticket to GitHub Support this afternoon. Let's see what they say !

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.

2 participants