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

add retry delay #2309

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

MaximilianKohler
Copy link
Contributor

I have no clue what I'm doing. It would be hilarious if this is correct and works.

I searched this: https://github.com/search?q=repo%3Aknadh%2Flistmonk%20max_msg_retries&type=code

And then duplicated it for "retry delay".

I got the email.go code from chat gpt.

Hopefully this closes #2272. I plan to set the delay to 60s+.

@knadh
Copy link
Owner

knadh commented Feb 17, 2025

This is almost correct, but there are syntax errors in the Go code in internal/messenger/email/email.go. I'll pick this up and fix in the coming weeks. The PR does save time, thanks!

@knadh knadh self-assigned this Feb 17, 2025
@MaximilianKohler
Copy link
Contributor Author

I asked Github Copilot and got a very different answer: https://github.com/copilot/share/02305230-4364-8806-b053-2c022449200c

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.

Retry delay or queue
2 participants