Configure sendmail: email throttling

JJSagan

Hi Devs,

My system shot too many emails (admittedly, it was my mistake), and I was flagged as a spammer
So I took a look at the code, specifically at src/emailer.js

Looks like Nodebb is using sendmail.
May I recommend changing line 24 as follows:

Emailer.transports = {
         sendmail: nodemailer.createTransport({
                 sendmail: true,
                 newline: 'unix',

                 pool: true, // use pooled connection
                 rateLimit: true, // enable to make sure we are limiting
                 maxConnections: 1, // set limit to 1 connection only
                 maxMessages: 1 // send 1 emails per second
        }),
}

Perhaps allow these settings to be configured from the ACP?

Kindly consider, thank you!
JJ.

yariplus

Please refer to the nodemailer documentation.

JJSagan

Hi @yariplus
I did, did I miss anything? It looks like the docs recommend setting the aforementioned parameters, however it seems that these parameters are not set in NodeBB.
Am I correct? If not kindly let me know.
Thanks again!
JJ.

PitaJ

@JJSagan I believe he was providing the reference for the developers.

As this is a feature request, you may want to open an issue on GitHub.

JJSagan

Hi @PitaJ ,
Done: https://github.com/NodeBB/NodeBB/issues/6911
In the meantime, is there a way to modify said js code without it being over-written on each update?