raise a RateLimitError when Twitter returns HTTP status 429 #837

2023-04-18T22:22:05-04:00

PrivacyDevel commented

2023-04-18 22:22:05 -04:00

(Migrated from github.com)

No description provided.

zedeus (Migrated from github.com) requested changes

2023-05-26 19:05:30 -04:00

src/apiutils.nim

				`@ -67,6 +67,9 @@ template fetchImpl(result, fetchBody) {.dirty.} =`

      if resp.status == $Http429:
        raise rateLimitError()

```suggestion if resp.status == $Http429: raise rateLimitError() ```

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin PrivacyDevel/429:PrivacyDevel/429

git switch PrivacyDevel/429

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master

git merge --no-ff PrivacyDevel/429

git switch PrivacyDevel/429

git rebase master

git switch master

git merge --ff-only PrivacyDevel/429

git switch PrivacyDevel/429

git rebase master

git switch master

git merge --no-ff PrivacyDevel/429

git switch master

git merge --squash PrivacyDevel/429

git switch master

git merge --ff-only PrivacyDevel/429

git switch master

git merge PrivacyDevel/429

git push origin master

Sign in to join this conversation.

No reviewers

zedeus

No labels

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

PAS/nitter!837

Rows
Columns