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.

Sign up for GitHub

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

Jump to bottom

Respect Retry-After for grabs #375

Open
bakerboy448 opened this issue Feb 23, 2023 · 1 comment · Fixed by #481
Open

Respect Retry-After for grabs #375

bakerboy448 opened this issue Feb 23, 2023 · 1 comment · Fixed by #481
Labels
feature New feature or request search Related to search

Comments

@bakerboy448
Copy link
Contributor

cross-seed/src/torznab.ts

Lines 431 to 436 in 7486f19

if (response.status === 429) {
updateIndexerStatus(
IndexerStatus.RATE_LIMITED,
Date.now() + ms("1 hour"),
[indexers[i].id]
);

rather than assuming an hour; check and use the retry-after header
@mynameisbogdan
Copy link
Contributor

Would be nice to respect the Retry-After header with a maximum value of 1 hour, for cases where the indexers rate limit per minute for example.

@mikewilzn mikewilzn mentioned this issue Feb 25, 2023
Closed
@bakerboy448 bakerboy448 mentioned this issue Sep 9, 2023
Merged
3 tasks
mmgoodnow added a commit that referenced this issue Sep 12, 2023
@mmgoodnow
Respect Retry-After header in Torznab queries (#481)
aa0d1f0 
# ToDo

- [x] cleanup initial chatgpt code 😅
- [ ] Test
- [ ] verbose logging?


# Git Ref
- Fixes #375
@mmgoodnow mmgoodnow reopened this Sep 12, 2023
@mmgoodnow mmgoodnow changed the title Implement Retry-After Respect Retry-After for grabs Sep 12, 2023
@zakkarry zakkarry added feature New feature or request search Related to search labels Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature or request search Related to search
Projects
Roadmap
Status: 📤 Ready
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Respect Retry-After header in Torznab queries bakerboy448/cross-seed
4 participants
@mynameisbogdan @mmgoodnow @bakerboy448 @zakkarry