search

zone-eu / zone-mta

📤 Modern outbound MTA cross platform and extendable server application
European Union Public License 1.2
599 stars 96 forks source link

onConnect should check for `this.closing` #309

Open titanism opened 2 years ago

titanism commented 2 years ago

Hi @andris9 - I think that your onConnect handler may also need the if (this.closing) return callback(new Error('Server is closing')). And shouldn't those (instances of if (this.closing) in your codebase) have err.responseCode = 421 as well?

titanism commented 2 years ago

Also you may need setImmediate in the onConnect as well.