New user -> no welcome mail sent to this new user

[erics]

Hello Cloudron,
something strange on our cloudron: when i create a new user and checkbox "send welcome message to this user" is cheched : no mail is sent ! (1)

then if i come back to user and clic on "reset password and invite link" mail is sent ... so is there a bug ? (2)

Note: i don't have any log for (1) in mail.log, do i have to check in other cloudron logs ?

thanks

[erics]

Ok,
here is the "bug", there is 6 administrators on my cloudron and new user send too many mails for embedded smtp server

2020-12-21T07:32:02.196Z box:mailer BoxError: Invalid greeting from server:
421 Too many concurrent connections: 421 Too many concurrent connections
    at /home/yellowtent/box/src/mailer.js:86:40
    at transporter.send.args (/home/yellowtent/box/node_modules/nodemailer/lib/mailer/index.js:226:21)
    at SMTPConnection.<anonymous> (/home/yellowtent/box/node_modules/nodemailer-smtp-transport/lib/smtp-transport.js:105:20)
    at Object.onceWrapper (events.js:286:20)
    at SMTPConnection.emit (events.js:198:13)
    at SMTPConnection._onError (/home/yellowtent/box/node_modules/smtp-connection/lib/smtp-connection.js:518:10)
    at SMTPConnection._actionGreeting (/home/yellowtent/box/node_modules/smtp-connection/lib/smtp-connection.js:893:14)
    at SMTPConnection._processResponse (/home/yellowtent/box/node_modules/smtp-connection/lib/smtp-connection.js:669:16)
    at SMTPConnection._onData (/home/yellowtent/box/node_modules/smtp-connection/lib/smtp-connection.js:493:10)
    at Socket.emit (events.js:198:13)

So @girish or @nebulon is there a solution to disable "new user mail notifications" to some administrators ? or to send invitation mail link to new user BEFORE administrators notifications ? or maybe add a little delay to mail send queue ?

[girish]

@erics said in New user -> no welcome mail sent to this new user:

Too many concurrent connections

Oh, that's not too good. Let me look into it. You can comment out this 1 line for now - https://git.cloudron.io/cloudron/box/-/blob/master/src/notifications.js#L120

The file in the server is /home/yellowtent/box/src/notifications.js and then systemctl restart box

        // mailer.userAdded(admin.email, user);

I will probably remove the email notifications altogether. They aren't exactly very useful since we raise a notification anyway and mostly just a carry over from our very old managed hosting code.

[erics]

@girish perfect, it works !