In OpenAccessNotifier#send_notification_to_user, we rescue Net::SMTPFatalError and log it so the iterator continues to the next notification. We have, however, run into other Net errors. Specifically: ReadTimeout. This could be caused by a temporary issue with the connection to the mail server, so I think it makes sense to rescue it as well, log it, and move to the next notification.
In
OpenAccessNotifier#send_notification_to_user, we rescueNet::SMTPFatalErrorand log it so the iterator continues to the next notification. We have, however, run into otherNeterrors. Specifically:ReadTimeout. This could be caused by a temporary issue with the connection to the mail server, so I think it makes sense to rescue it as well, log it, and move to the next notification.