From 20ab78182a147e4194a6ed7ea3da9f9836155d09 Mon Sep 17 00:00:00 2001
From: Jorge Miguel Lobo Escalona
 <47326048+jloboescalona2@users.noreply.github.com>
Date: Fri, 7 May 2021 10:31:04 +0200
Subject: [PATCH] M #-: Remove "Token expired" in fireedge.log (#1192)

Signed-off-by: Jorge Lobo <jlobo@opennebula.io>
---
 src/fireedge/src/server/utils/jwt.js | 56 ++++++++++++++--------------
 1 file changed, 29 insertions(+), 27 deletions(-)

diff --git a/src/fireedge/src/server/utils/jwt.js b/src/fireedge/src/server/utils/jwt.js
index c31aa79430..f254219fc6 100644
--- a/src/fireedge/src/server/utils/jwt.js
+++ b/src/fireedge/src/server/utils/jwt.js
@@ -14,7 +14,6 @@
 /* -------------------------------------------------------------------------- */
 
 const jwt = require('jwt-simple')
-const { DateTime } = require('luxon')
 const { messageTerminal } = require('./general')
 
 const createToken = (
@@ -42,35 +41,38 @@ const validateAuth = req => {
     const authorization = req.headers.authorization
     const removeBearer = new RegExp('^Bearer ', 'i')
     const token = authorization.replace(removeBearer, '')
-    try {
-      const payload = jwt.decode(token, global.FIREEDGE_KEY)
-      const now = DateTime.local()
-      if (
-        payload &&
-        'iss' in payload &&
-        'aud' in payload &&
-        'jti' in payload &&
-        'iat' in payload &&
-        'exp' in payload &&
-        payload.exp >= now.toSeconds()
-      ) {
-        const { iss, aud, jti, iat, exp } = payload
-        rtn = {
-          iss,
-          aud,
-          jti,
-          iat,
-          exp
+    const fireedgeKey = global && global.FIREEDGE_KEY
+    if (token && fireedgeKey) {
+      try {
+        const payload = jwt.decode(token, fireedgeKey)
+        if (
+          payload &&
+          'iss' in payload &&
+          'aud' in payload &&
+          'jti' in payload &&
+          'iat' in payload &&
+          'exp' in payload
+        ) {
+          const { iss, aud, jti, iat, exp } = payload
+          rtn = {
+            iss,
+            aud,
+            jti,
+            iat,
+            exp
+          }
         }
+      } catch (error) {
       }
-    } catch (error) {
-      const config = {
-        color: 'red',
-        type: 'ERROR',
-        error: (error && error.message) || '',
-        message: 'Error: %s'
+    } else {
+      const messageError = (!token && 'jwt') || (!fireedgeKey && 'fireedge_key')
+      if (messageError) {
+        messageTerminal({
+          color: 'red',
+          message: 'invalid: %s',
+          type: messageError
+        })
       }
-      messageTerminal(config)
     }
   }
   return rtn