From 732afd83933735162a36b44d2419f73a998d2afc Mon Sep 17 00:00:00 2001 From: Maycon Santos Date: Mon, 25 Sep 2023 07:57:16 +0200 Subject: [PATCH] Revert zitadel update parameters endpoint (#1163) * Revert zitadel update parameters endpoint With previous release we broke the parameters' endpoint. This Pr reverses that * add error log to util --- management/server/http/util/util.go | 1 + management/server/idp/zitadel.go | 5 +++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/management/server/http/util/util.go b/management/server/http/util/util.go index 44f4919f5..277627310 100644 --- a/management/server/http/util/util.go +++ b/management/server/http/util/util.go @@ -77,6 +77,7 @@ func WriteErrorResponse(errMsg string, httpStatus int, w http.ResponseWriter) { // WriteError converts an error to an JSON error response. // If it is known internal error of type server.Error then it sets the messages from the error, a generic message otherwise func WriteError(err error, w http.ResponseWriter) { + log.Errorf("got a handler error: %s", err.Error()) errStatus, ok := status.FromError(err) httpStatus := http.StatusInternalServerError msg := "internal server error" diff --git a/management/server/idp/zitadel.go b/management/server/idp/zitadel.go index 73958a69e..e42b9a506 100644 --- a/management/server/idp/zitadel.go +++ b/management/server/idp/zitadel.go @@ -13,8 +13,9 @@ import ( "time" "github.com/golang-jwt/jwt" - "github.com/netbirdio/netbird/management/server/telemetry" log "github.com/sirupsen/logrus" + + "github.com/netbirdio/netbird/management/server/telemetry" ) // ZitadelManager zitadel manager client instance. @@ -428,7 +429,7 @@ func (zm *ZitadelManager) UpdateUserAppMetadata(userID string, appMetadata AppMe return err } - resource := fmt.Sprintf("users/%s", userID) + resource := fmt.Sprintf("users/%s/metadata/_bulk", userID) _, err = zm.post(resource, string(payload)) if err != nil { return err