From 24310c63e2328d415b3927a73c797f2fa08bbee8 Mon Sep 17 00:00:00 2001 From: Zoltan Papp Date: Tue, 7 Mar 2023 15:01:47 +0100 Subject: [PATCH] Remove mgm close steps, in defer doing it already (#729) Simple code cleaning. Remove duplicated steps in login. In the defer already close the management connection. --- client/internal/login.go | 7 ------- 1 file changed, 7 deletions(-) diff --git a/client/internal/login.go b/client/internal/login.go index 25e5801db..677ff1edc 100644 --- a/client/internal/login.go +++ b/client/internal/login.go @@ -59,13 +59,6 @@ func Login(ctx context.Context, config *Config, setupKey string, jwtToken string return err } log.Infof("peer has successfully logged-in to the Management service %s", config.ManagementURL.String()) - - err = mgmClient.Close() - if err != nil { - log.Errorf("failed to close the Management service client: %v", err) - return err - } - return nil }