diff options
| author | jwijenbergh <jeroenwijenbergh@protonmail.com> | 2024-03-13 16:37:24 +0100 |
|---|---|---|
| committer | jwijenbergh <jeroenwijenbergh@protonmail.com> | 2024-03-13 16:37:24 +0100 |
| commit | 44a06427f1d3a5c088f9c0dc0ec813a5db227888 (patch) | |
| tree | 3f1ced7444b647553d67e2c5a3901abc2e6eee52 /internal/server/secureinternet.go | |
| parent | a92f6888bec213ed81c327ac24461266ac0b420b (diff) | |
Server: Log on remove after failed authorization error
Diffstat (limited to 'internal/server/secureinternet.go')
| -rw-r--r-- | internal/server/secureinternet.go | 6 |
1 files changed, 5 insertions, 1 deletions
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 |