From c9ae63b4aee338e75048aaec73378c77be785ff5 Mon Sep 17 00:00:00 2001 From: Douglas Barone Date: Tue, 19 Jan 2021 08:59:55 -0400 Subject: [PATCH] Improved log messages --- server/src/classes/ResetToken.js | 2 +- server/src/classes/User.js | 15 ++++++++------- server/src/cronTasks.js | 6 +++--- server/src/index.js | 6 +++--- server/src/lib/activeDirectory/passwordUtils.js | 2 +- server/src/lib/paloalto.js | 6 ++++-- server/src/lib/wifiDevices.js | 4 ++-- server/src/resolvers/Mutation/index.js | 2 +- server/src/tasks.js | 2 +- 9 files changed, 24 insertions(+), 21 deletions(-) diff --git a/server/src/classes/ResetToken.js b/server/src/classes/ResetToken.js index 1595701..2f01d0d 100755 --- a/server/src/classes/ResetToken.js +++ b/server/src/classes/ResetToken.js @@ -106,7 +106,7 @@ class ResetToken { logSuccess({ tags: ['token', 'user'], - message: `User ${user.sAMAccountName} used a reset token` + message: `Usuário ${user.sAMAccountName} utilizou um token.` }) return true diff --git a/server/src/classes/User.js b/server/src/classes/User.js index 3bbd527..7553b21 100755 --- a/server/src/classes/User.js +++ b/server/src/classes/User.js @@ -141,7 +141,7 @@ class User { client.on('error', err => { client.unbind(() => { logError({ - message: 'Client unbinded due error.', + message: 'Cliente LDAP desconectado devido à um erro.', data: err, tags: ['ldap'] }) @@ -192,7 +192,8 @@ class User { if (err) { client.unbind(() => { logError({ - message: 'Client unbinded. Error updating password', + message: + 'Cliente LDAP desconectado devido à um erro ao atualizar uma senha.', data: err, tags: ['ldap'] }) @@ -202,7 +203,7 @@ class User { User.upsertUser(this.username) client.unbind(() => { logSuccess({ - message: `User ${this.username} updated his password`, + message: `Usuário ${this.username} alterou sua senha.`, data: result, tags: ['ldap'] }) @@ -286,26 +287,26 @@ class User { try { logInfo({ tags: ['user', 'login'], - message: `Usuáio ${username} está tentando logar` + message: `Usuário ${username} está tentando logar` }) await ad.authenticate(`ifms\\${username}`, password) logSuccess({ tags: ['user', 'login'], - message: `Usuário ${username} logou com sucesso` + message: `Usuário ${username} logou com sucesso.` }) } catch (err) { if (await ad.checkBinding()) { logWarning({ tags: ['user', 'login', 'password'], - message: `Usuário ${username} tentou logar, mas falhou` + message: `Usuário ${username} tentou logar, mas falhou.` }) throw new Error('Usuário ou senha inválidos.') } else { logError({ tags: ['user', 'login', 'password'], - message: `Can't bind with AD.`, + message: `Não é possível conectar ao AD.`, data: err }) throw new Error('Problemas técnicos ao autenticar. Tente mais tarde.') diff --git a/server/src/cronTasks.js b/server/src/cronTasks.js index 5f34ef7..040e626 100644 --- a/server/src/cronTasks.js +++ b/server/src/cronTasks.js @@ -12,14 +12,14 @@ logInfo({ cron.schedule('0 0 0 * * *', async () => { logInfo({ tags: ['cron', 'user'], - message: `User.importAllUsers iniciado` + message: `Importação dos usuários do AD iniciada.` }) const users = await User.importAllUsers() logInfo({ tags: ['cron', 'user'], - message: `User.importAllUsers importou ${users} usuários` + message: `Foram importados ${users} usuários do AD.` }) }) @@ -35,6 +35,6 @@ cron.schedule('0 0 0 * * *', async () => { await deleteOldLogs() logSuccess({ tags: ['cron', 'log'], - message: `Old logs deleted` + message: `Logs antigos deletados` }) }) diff --git a/server/src/index.js b/server/src/index.js index e96fec9..7e15750 100755 --- a/server/src/index.js +++ b/server/src/index.js @@ -13,14 +13,14 @@ import './cronTasks' server.listen().then(options => { logSuccess({ tags: ['server'], - message: `Server ready!`, + message: `Servidor pronto!`, data: { endpoint: `${options.url}graphql`, websocket: `${options.subscriptionsUrl}`, env: process.env.NODE_ENV === 'production' - ? 'Running in production' - : 'Running in development' + ? 'Rodando em modo de produção' + : 'Rodando em modo de desenvolvimento' } }) }) diff --git a/server/src/lib/activeDirectory/passwordUtils.js b/server/src/lib/activeDirectory/passwordUtils.js index 5302bac..c2042f7 100755 --- a/server/src/lib/activeDirectory/passwordUtils.js +++ b/server/src/lib/activeDirectory/passwordUtils.js @@ -57,7 +57,7 @@ const replacePassword = (username, newPassword) => { User.upsertUser(username) logSuccess({ tags: ['user', 'replacePassword'], - message: `Senha atualizada para o usuário ${username}`, + message: `Senha alterada para o usuário ${username}`, data: result }) resolve('Senha alterada com sucesso!') diff --git a/server/src/lib/paloalto.js b/server/src/lib/paloalto.js index 8c52702..0a43526 100644 --- a/server/src/lib/paloalto.js +++ b/server/src/lib/paloalto.js @@ -83,7 +83,7 @@ async function updateUserIdMappings() { logSuccess({ tags: ['user-id', 'paloalto'], - message: `Atualizado ${devices.length} mapeamentos de user-id no ${ + message: `Foram mapeados ${devices.length} user-ids em ${ pAHost.description || pAHost.cidr }`, data: devices @@ -93,7 +93,9 @@ async function updateUserIdMappings() { } catch (e) { logError({ tags: ['paloalto', 'user-id'], - message: `Erro atualizando user-id mappings: ${e.message}`, + message: `Erro atualizando user-id mappings em ${ + pAHost.description || pAHost.cidr + }: ${e.message}`, data: e }) diff --git a/server/src/lib/wifiDevices.js b/server/src/lib/wifiDevices.js index 569751e..e980953 100644 --- a/server/src/lib/wifiDevices.js +++ b/server/src/lib/wifiDevices.js @@ -123,7 +123,7 @@ async function updateDB(onlineDevices) { else logError({ tags: ['wifiDevices'], - message: `Erro tentando adicionar o dispositivo "${device.mac}"`, + message: `Erro tentando adicionar o dispositivo "${device.mac}".`, data: { error: e, device } }) } @@ -148,7 +148,7 @@ async function updateDevicesInfo() { logSuccess({ tags: ['wifiDevices'], - message: `Atualizados ${onlineDevices.length} dispositivos`, + message: `Foram atualizados ${onlineDevices.length} dispositivos Wi-Fi.`, data: onlineDevices }) diff --git a/server/src/resolvers/Mutation/index.js b/server/src/resolvers/Mutation/index.js index 45a4c30..908a77f 100644 --- a/server/src/resolvers/Mutation/index.js +++ b/server/src/resolvers/Mutation/index.js @@ -64,7 +64,7 @@ const Mutation = { if (host) logSuccess({ - message: `User ${auth.sAMAccountName}(${auth.displayName}) adicionou um novo host Palo Alto (${host.cidr}).`, + message: `Usuário ${auth.sAMAccountName} (${auth.displayName}) adicionou um novo host Palo Alto (${host.cidr}).`, tags: ['paloalto'] }) diff --git a/server/src/tasks.js b/server/src/tasks.js index 5210452..04717ad 100644 --- a/server/src/tasks.js +++ b/server/src/tasks.js @@ -8,7 +8,7 @@ async function updateDevicesTask() { try { logInfo({ tags: ['task', 'wifiDevices'], - message: 'updateDevicesTask iniciou' + message: 'Atualização de dispositivos iniciou.' }) const devicesQnt = await updateDevicesInfo()