Proposal to allow user to OTP if mailing fails in development

[kylerwsm]

Problem

In development mode, we are using nodemailer-direct-transport to mail OTPs. But it does not provide consistent mailing performance across different IP addresses. Some IPs addresses may be blacklisted from outbound mailing resulting in errors like this. In these cases, developers may not be able to proceed to the OTP and user pages without making code changes.

Since we have a default OTP for local development, and nodemailer-direct-transport is a local development only issue, I thought that creating allowing users to proceed to OTP even if mailing fails in development may be appropriate and helpful.

Solution

When there is a mail error, I added a conditional statement that checks if the environment is a development environment. If it is, the server responses with an ok status after printing some relevant warning logs.