From cec330c1644a0318f2037b786b4cb3a34c6f17ff Mon Sep 17 00:00:00 2001 From: jwijenbergh Date: Thu, 17 Nov 2022 17:11:30 +0100 Subject: Log: Add additional params to get rid of Sprintf calls --- client/server.go | 32 +++++++++++--------------------- 1 file changed, 11 insertions(+), 21 deletions(-) (limited to 'client/server.go') diff --git a/client/server.go b/client/server.go index 5e1c31f..3bcecc6 100644 --- a/client/server.go +++ b/client/server.go @@ -83,7 +83,7 @@ func (client *Client) getConfig( // This is best effort endpointErr := server.RefreshEndpoints(chosenServer) if endpointErr != nil { - client.Logger.Warning(fmt.Sprintf("failed to refresh server endpoints: %v", endpointErr)) + client.Logger.Warning("failed to refresh server endpoints: %v", endpointErr) } config, configType, configErr := client.retryConfigAuth(chosenServer, preferTCP) @@ -103,10 +103,8 @@ func (client *Client) getConfig( saveErr := client.Config.Save(&client) if saveErr != nil { client.Logger.Info( - fmt.Sprintf( - "Failed saving configuration after getting a server: %s", - types.GetErrorTraceback(saveErr), - ), + "Failed saving configuration after getting a server: %s", + types.GetErrorTraceback(saveErr), ) } @@ -154,10 +152,8 @@ func (client *Client) RemoveSecureInternet() error { saveErr := client.Config.Save(&client) if saveErr != nil { client.Logger.Info( - fmt.Sprintf( - "Failed saving configuration after removing a secure internet server: %s", - types.GetErrorTraceback(saveErr), - ), + "Failed saving configuration after removing a secure internet server: %s", + types.GetErrorTraceback(saveErr), ) } return nil @@ -180,10 +176,8 @@ func (client *Client) RemoveInstituteAccess(url string) error { saveErr := client.Config.Save(&client) if saveErr != nil { client.Logger.Info( - fmt.Sprintf( - "Failed saving configuration after removing an institute access server: %s", - types.GetErrorTraceback(saveErr), - ), + "Failed saving configuration after removing an institute access server: %s", + types.GetErrorTraceback(saveErr), ) } return nil @@ -206,10 +200,8 @@ func (client *Client) RemoveCustomServer(url string) error { saveErr := client.Config.Save(&client) if saveErr != nil { client.Logger.Info( - fmt.Sprintf( - "Failed saving configuration after removing a custom server: %s", - types.GetErrorTraceback(saveErr), - ), + "Failed saving configuration after removing a custom server: %s", + types.GetErrorTraceback(saveErr), ) } return nil @@ -569,10 +561,8 @@ func (client *Client) ShouldRenewButton() bool { if currentServerErr != nil { client.Logger.Info( - fmt.Sprintf( - "No server found to renew with err: %s", - types.GetErrorTraceback(currentServerErr), - ), + "No server found to renew with err: %s", + types.GetErrorTraceback(currentServerErr), ) return false } -- cgit v1.2.3