From 44a06427f1d3a5c088f9c0dc0ec813a5db227888 Mon Sep 17 00:00:00 2001 From: jwijenbergh Date: Wed, 13 Mar 2024 16:37:24 +0100 Subject: Server: Log on remove after failed authorization error --- internal/server/secureinternet.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'internal/server/secureinternet.go') diff --git a/internal/server/secureinternet.go b/internal/server/secureinternet.go index 3377650..4b9c29e 100644 --- a/internal/server/secureinternet.go +++ b/internal/server/secureinternet.go @@ -7,6 +7,7 @@ import ( "github.com/eduvpn/eduvpn-common/internal/api" "github.com/eduvpn/eduvpn-common/internal/config/v2" "github.com/eduvpn/eduvpn-common/internal/discovery" + "github.com/eduvpn/eduvpn-common/internal/log" "github.com/eduvpn/eduvpn-common/internal/util" "github.com/eduvpn/eduvpn-common/types/server" "github.com/jwijenbergh/eduoauth-go" @@ -54,7 +55,10 @@ func (s *Servers) AddSecure(ctx context.Context, disco *discovery.Discovery, org _, err = api.NewAPI(ctx, s.clientID, sd, s.cb, nil) if err != nil { // authorization has failed, remove the server again - s.config.RemoveServer(orgID, server.TypeSecureInternet) + rerr := s.config.RemoveServer(orgID, server.TypeSecureInternet) + if rerr != nil { + log.Logger.Warningf("could not remove secure internet server: '%s' after failing authorization: %v", orgID, rerr) + } return err } return nil -- cgit v1.2.3