From 1ab330de602e2bf74b14c42e21bd7dcdaaa2d3b6 Mon Sep 17 00:00:00 2001
From: Douglas Barone <douglasvbarone@gmail.com>
Date: Fri, 15 Jan 2021 07:03:47 -0400
Subject: [PATCH] Improve logging

---
 server/src/classes/User.js | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/server/src/classes/User.js b/server/src/classes/User.js
index 7b228b1..1df89fa 100755
--- a/server/src/classes/User.js
+++ b/server/src/classes/User.js
@@ -350,7 +350,7 @@ class User {
   static async importAllUsers() {
     if (this.working) {
       logWarning({
-        tags: ['user'],
+        tags: ['user', 'AD'],
         message: 'User import aborted: already working'
       })
       return 0
@@ -360,8 +360,8 @@ class User {
 
     try {
       logInfo({
-        tags: ['user'],
-        message: 'Importing users from Active Directory'
+        tags: ['user', 'AD'],
+        message: 'Getting users from Active Directory.'
       })
 
       const allAdUsers = await ad.findUsers({
@@ -370,8 +370,8 @@ class User {
       })
 
       logInfo({
-        tags: ['user'],
-        message: 'Importing users to database'
+        tags: ['user', 'AD'],
+        message: 'Importing users into database'
       })
 
       await Promise.all(
@@ -387,12 +387,16 @@ class User {
       )
 
       logSuccess({
-        tags: ['user'],
-        message: `${allAdUsers.length} usuários importados`
+        tags: ['user', 'AD'],
+        message: `${allAdUsers.length} usuários importados do Active Directory`
       })
 
       return allAdUsers.length
     } catch (e) {
+      logError({
+        message: `Erro ao importar usuários. ${e.message}`,
+        tags: ['user', 'AD']
+      })
       throw new Error('Erro ao importar usuários: ' + e)
     } finally {
       this.working = false