search

pmclanahan / django-celery-email

A Django email backend that uses a celery task for sending the email.
BSD 3-Clause "New" or "Revised" License
477 stars 110 forks source link

Retry on connection open failures #63

Open joshbuddy opened 5 years ago

joshbuddy commented 5 years ago

When hitting an error on connection.open, we probably prefer to retry with some backoff instead of raising another exception. It's a bit tricky to test, just wanted to run the idea by y'all first.

pmac commented 4 years ago

@joshbuddy do you think this is ready to go out? A test to make sure the retry exception is raised wouldn't hurt, but it's probably not that big a deal. We'll need to rebase as we just merged #66 . I can also release a version for pre 3.0.0 that would support older Djangos if anyone needs that.