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

bug(discord): hits rate-limit with high load #105

Closed
vember31 opened this issue Nov 14, 2024 · 3 comments · Fixed by #114
Closed

bug(discord): hits rate-limit with high load #105

vember31 opened this issue Nov 14, 2024 · 3 comments · Fixed by #114
Labels
bug Something isn't working

Comments

@vember31
Copy link

vember31 commented Nov 14, 2024

Description of the bug

Back with another one 😄 thanks for going through all of these @dan-online !

I'm getting the below error, it seems usually when I've downloaded a season of a show in sonarr. when i download an individual episode from sonarr, that seems to work, but get the below error with season downloads.

image

if helpful, my sonarr settings are the following:

image

Do we know what would cause this error?

Steps To Reproduce

  1. Download a season of a show in sonarr
  2. Ensure it pushes to autopulse upon file import
  3. No notification comes through & I see the above error in the logs

Additional Information

There are no other issues - everything loads up into autopulse just fine and it processes all of the files appropriately! Just the notification that fails to send.

@vember31 vember31 added the bug Something isn't working label Nov 14, 2024
@dan-online
Copy link
Owner

Seems like this is either a ratelimit on Discord's end or the embed description is too long although autopulse should group together batches. I'll check this out either way but it's probably the latter case

@dan-online
Copy link
Owner

Hey @vember31 feel free try out pr-114 and see if that helps

@dan-online dan-online changed the title bug: discord notification fails on season import from a tv show bug(discord): hits rate-limit with high load Dec 2, 2024
@vember31
Copy link
Author

vember31 commented Dec 5, 2024

Should've put a comment a bit back but this is fixed as far as I can tell over the past couple of weeks! I haven't run into this issue since any of the changes here from #114. I have downloaded one (somewhat small) season since getting the fix and it sent notifications as expected, and all were bundled together as well.

Going to close this - thank you again @dan-online!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants