Address some linter warnings

Response bodies need to be closed
This commit is contained in:
Ralf Haferkamp
2023-03-20 12:36:55 +01:00
committed by Ralf Haferkamp
parent 7f8e54b1c2
commit 98b4a6a23a

View File

@@ -155,6 +155,8 @@ func (c *cs3backend) CreateUserFromClaims(ctx context.Context, claims map[string
req := lgClient.UsersApi.CreateUser(newctx).User(newUser)
created, resp, err := req.Execute()
defer resp.Body.Close()
var reread bool
if err != nil {
if resp == nil {
@@ -178,7 +180,8 @@ func (c *cs3backend) CreateUserFromClaims(ctx context.Context, claims map[string
if reread {
c.logger.Debug().Msg("User already exist, re-reading via libregraph")
gureq := lgClient.UserApi.GetUser(newctx, newUser.GetOnPremisesSamAccountName())
created, _, err = gureq.Execute()
created, resp, err = gureq.Execute()
defer resp.Body.Close()
if err != nil {
c.logger.Error().Err(err).Msg("Error trying to re-read user from graphAPI")
return nil, err