Fix system mailer, mail logging

bin/www.js

@@ -67,12 +67,12 @@ function onError(error) {
     case 'EACCES':
       console.error(bind + ' requires elevated privileges')
       logger.error(
-        `server.www.startup__Specified port requires elevated privileges.`
+        'server.www.startup__Specified port requires elevated privileges.'
       )
       process.exit(1)
     case 'EADDRINUSE':
       console.error(bind + ' is already in use')
-      logger.error(`server.www.startup__Specified port is already in use.`)
+      logger.error('server.www.startup__Specified port is already in use.')
       process.exit(1)
     default:
       throw error

functions/sendMail.js

@@ -1,8 +1,12 @@
 import { createTransport } from 'nodemailer'
+import logger from './logger.js'
 
 export function sendMail(mailto, mailsubject, mailbody, image) {
   // In case mail is destined for system administrator or we run in development environment, send all e-mails to system address obtained from config.
   if (process.env.NODE_ENV === 'development' || mailto === 'system@system') {
+    logger.warn(
+      `server.functions.sendmail__Sending e-mail [${mailsubject}] to administrator address.`
+    )
     mailto = process.env.MAIL_SYSTEM || 'root@localhost'
   }
 
@@ -37,7 +41,22 @@ export function sendMail(mailto, mailsubject, mailbody, image) {
     ]
   }
 
-  transporter.sendMail(mailOptions, function (error, info) {
-    if (error) console.log(error)
-  })
+  transporter
+    .sendMail(mailOptions)
+    .then((info) => {
+      logger.info(
+        `server.functions.sendmail__Succesfully sent e-mail [${JSON.stringify(
+          mailOptions
+        )}].`,
+        info
+      )
+    })
+    .catch((err) => {
+      logger.error(
+        `server.functions.sendmail__Failed to send e-mail [${JSON.stringify(
+          mailOptions
+        )}]. Error:`,
+        err
+      )
+    })
 }

routes/api/keypadOrder.js

@@ -153,7 +153,7 @@ router.post('/', ensureAuthenticatedAPI, function (req, res, next) {
                 .catch((err) => {
                   const subject = 'Nepodařilo se zapsat změny do databáze!'
                   const body = `<h1>Chyba při zapisování do databáze při nákupu!</h1><p>Pokus o vytvoření záznamu nákupu skončil chybou. Zkontrolujte konzistenci databáze!</p><p>Chyba: ${err.message}</p>`
-                  sendMail('system', subject, body)
+                  sendMail('system@system', subject, body)
                   return
                 })
             }

routes/invoices.js

@@ -96,7 +96,7 @@ router.post('/', ensureAuthenticated, function (req, res, _next) {
     if (!check.buyerId.equals(req.user.id)) {
       const subject = 'Neoprávněná akce?!'
       const body = `<h1>Jak se toto podařilo?!</h1><p>Zákazník ${req.body.displayName} se pokouší manipulovat s fakturou ID ${check._id}, přestože ji nevlastní.</p>Jeho akce byla revertována. Prověřte celou situaci!</p>`
-      sendMail('system', subject, body)
+      sendMail('system@system', subject, body)
       const alert = {
         type: 'danger',
         message: 'Nemáte oprávnění měnit status faktury, která Vám nepatří!',

routes/kiosk_shop.js

@@ -205,7 +205,7 @@ router.post('/', ensureAuthenticated, function (req, res) {
                   res.redirect('/kiosk_keypad')
                   const subject = 'Nepodařilo se zapsat změny do databáze!'
                   const body = `<h1>Chyba při zapisování do databáze při nákupu!</h1><p>Pokus o vytvoření záznamu nákupu skončil chybou. Zkontrolujte konzistenci databáze!</p><p>Chyba: ${err.message}</p>`
-                  sendMail('system', subject, body)
+                  sendMail('system@system', subject, body)
                   return
                 })
             })
@@ -219,7 +219,7 @@ router.post('/', ensureAuthenticated, function (req, res) {
               res.redirect('/kiosk_keypad')
               const subject = 'Nepodařilo se zapsat změny do databáze!'
               const body = `<h1>Chyba při zapisování do databáze při nákupu!</h1><p>Pokus o snížení skladové zásoby skončil chybou. Zkontrolujte konzistenci databáze!</p><p>Chyba: ${err.message}</p>`
-              sendMail('system', subject, body)
+              sendMail('system@system', subject, body)
               return
             })
         })

routes/payments.js

@@ -115,7 +115,7 @@ router.post('/', ensureAuthenticated, function (req, res, _next) {
     if (!check.supplierId.equals(req.user.id)) {
       var subject = 'Neoprávněná akce?!'
       var body = `<h1>Jak se toto podařilo?!</h1><p>Dodavatel ${req.body.displayName} se pokouší manipulovat s fakturou ID ${check._id}, přestože ji nevytvořil.</p>Jeho akce byla revertována. Prověřte celou situaci!</p>`
-      sendMail('system', subject, body)
+      sendMail('system@system', subject, body)
       const alert = {
         type: 'danger',
         message:

routes/shop.js

@@ -155,7 +155,7 @@ router.post('/', ensureAuthenticated, checkKiosk, function (req, res) {
               res.redirect('/shop')
               const subject = 'Nepodařilo se zapsat změny do databáze!'
               const body = `<h1>Chyba při zapisování do databáze při nákupu!</h1><p>Pokus o vytvoření záznamu nákupu skončil chybou. Zkontrolujte konzistenci databáze!</p><p>Chyba: ${err.message}</p>`
-              sendMail('system', subject, body)
+              sendMail('system@system', subject, body)
               return
             })
         })
@@ -170,7 +170,7 @@ router.post('/', ensureAuthenticated, checkKiosk, function (req, res) {
           res.redirect('/shop')
           const subject = 'Nepodařilo se zapsat změny do databáze!'
           const body = `<h1>Chyba při zapisování do databáze při nákupu!</h1><p>Pokus o snížení skladové zásoby skončil chybou. Zkontrolujte konzistenci databáze!</p><p>Chyba: ${err.message}</p>`
-          sendMail('system', subject, body)
+          sendMail('system@system', subject, body)
           return
         })
     })