From a27fe4326c32be4a44ea2b415bb787a65ec851d3 Mon Sep 17 00:00:00 2001 From: Maycon Santos Date: Thu, 23 Mar 2023 18:26:41 +0100 Subject: [PATCH] Add JWT middleware validation failure log (#760) We will log the middleware log now, but in the next releases we should provide a generic error that can be parsed by the dashboard. --- management/server/http/middleware/jwt.go | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/management/server/http/middleware/jwt.go b/management/server/http/middleware/jwt.go index feb00ec86..1ac6d3948 100644 --- a/management/server/http/middleware/jwt.go +++ b/management/server/http/middleware/jwt.go @@ -4,12 +4,14 @@ import ( "context" "errors" "fmt" - "github.com/golang-jwt/jwt" - "github.com/netbirdio/netbird/management/server/http/util" - "github.com/netbirdio/netbird/management/server/status" - "log" "net/http" "strings" + + "github.com/golang-jwt/jwt" + log "github.com/sirupsen/logrus" + + "github.com/netbirdio/netbird/management/server/http/util" + "github.com/netbirdio/netbird/management/server/status" ) // A function called whenever an error is encountered @@ -114,6 +116,9 @@ func (m *JWTMiddleware) Handler(h http.Handler) http.Handler { // If there was an error, do not continue. if err != nil { + log.Errorf("received an error while validating the JWT token: %s. "+ + "Review your IDP configuration and ensure that "+ + "settings are in sync between dashboard and management", err) return }