sct / overseerr

Request management and media discovery tool for the Plex ecosystem
https://overseerr.dev
MIT License
3.71k stars 435 forks source link

[Feature Request] A "retry" option for notifications (SMTP), if it fails #3659

Open ghost opened 10 months ago

ghost commented 10 months ago

Description

Pretty much as the title says.

It would be nice to have a "Retry n times", if the SMTP server fails for whatever reason.

Sometimes the SMTP server I use give a connect ECONNREFUSED 5.XXX.XXX.58:587 error, which is usually just a temporary error that lasts shortly.

It would be a nice little feature, since there seems to be no option for this as of now :).

Desired Behavior

1) User sets "Retry n times" to e.g. 5 2) The SMTP notification fails - with e.g. connect ECONNREFUSED 5.XXX.XXX.58:587 2) Jellyseerr now retries it 5 times, with a short delay for each time, 5 seconds or n (user definable?) 3) If notification gets sent succesfully (no error), the retrying stops 4) If notification still fails after n retries, output it to logs

Additional Context

Also made in the Jellyseerr repo: https://github.com/Fallenbagel/jellyseerr/issues/507

Code of Conduct

stale[bot] commented 5 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

engels74 commented 5 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Please no stale :)