yesBad/proton-bridge
1
0
Fork 0
forked from Silverfish/proton-bridge
Code Actions Activity

feat: max retries of 5 for client logout

Browse Source
This commit is contained in:
James Houlahan
2020-04-17 15:39:02 +02:00
parent e95aece6d3
commit 99635cd56d
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
pkg/pmapi/clientmanager.go
View File

@ -11,6 +11,8 @@ import (
"github.com/sirupsen/logrus" "github.com/sirupsen/logrus"
) )
const maxLogoutRetries = 5
// ClientManager is a manager of clients. // ClientManager is a manager of clients.
type ClientManager struct { type ClientManager struct {
// newClient is used to create new Clients. By default this creates pmapi clients but it can be overridden to // newClient is used to create new Clients. By default this creates pmapi clients but it can be overridden to
@ -153,8 +155,17 @@ func (cm *ClientManager) LogoutClient(userID string) {
return return
} }
var retries int
for client.DeleteAuth() == ErrAPINotReachable { for client.DeleteAuth() == ErrAPINotReachable {
cm.log.Warn("Logging out client failed because API was not reachable, retrying...") retries++
if retries > maxLogoutRetries {
cm.log.Error("Failed to delete client auth (retried too many times)")
break
}
cm.log.Warn("Failed to delete client auth because API was not reachable, retrying...")
} }
}() }()
} }