Teiserver swallows all email sending errors

[p2004a]

If sending emails fails, there isn't any error returned to user or log message created.

Calls to Teiserver.Mailer.deliver_now aren't checking the return tuple, and code isn't written to handle failures in sending of the emails: e.g. reset code is created before sending email and not cleaned up if it fails, so if email sending fails it's not logged, user code is still created blocking new recovery codes creation, and user has have to wait 24h to retry sending password link.

[StanczakDominik]

Closed by #290

[p2004a]

@StanczakDominik Is that also handling registration?

[StanczakDominik]

I'll double check

[geekingfrog]

I did a check locally, and it only logs an error, there's no feedback for chobby.
The server logs the error:

[error] Error sending new user email - <THE-EMAIL> - {:network_failure, '127.0.0.1', {:error, :econnrefused}}

but the code is still in the database.

I created: #310