Merge pull request #777 from telefonicaid/task/release_transport_reso…

…urces

release nodemailer transport resources

CHANGES_NEXT_RELEASE

@@ -1,2 +1,3 @@
+- Fix: release transport resources of nodemailer after send email
 - Upgrade express dep from 4.18.1 to 4.19.2 due to a vulnerability
 - Upgrade nodemailer dep from 6.6.5 to 6.9.13 due to a vulnerability

lib/models/emailAction.js

@@ -64,13 +64,13 @@ function buildMailOptions(action, event) {
 }
 
 function SendMail(action, event, callback) {
+    var transporter;
     try {
         // setup e-mail data with unicode symbols
         var mailOptions = buildMailOptions(action, event),
             transporterOptions = buildTransporterOptions(action, event),
             opt2log,
             smtpConfig,
-            transporter,
             msgFromConfig;
 
         if (transporterOptions.smtp) {
@@ -119,6 +119,11 @@ function SendMail(action, event, callback) {
         // Not an HTTP request, so outgoingTransacion hasn't already counted and must be counted now
         metrics.IncMetrics(event.service, event.subservice, metrics.outgoingTransactionsErrors);
         return callback(ex);
+    } finally {
+        if (transporter) {
+            // release resources
+            transporter.close();
+        }
     }
 }