From 595a6580f5c24780b5a3fd8c3769e3f3ba3c3411 Mon Sep 17 00:00:00 2001 From: Ghostkeeper Date: Mon, 22 Nov 2021 16:17:30 +0100 Subject: [PATCH] Use error callback and with correct number of parameters It gives two parameters, additionally an error code. However our callback wrapper gets the error code from the reply itself so it doesn't need it. Contributes to issue CURA-8539. --- cura/OAuth2/AuthorizationHelpers.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/cura/OAuth2/AuthorizationHelpers.py b/cura/OAuth2/AuthorizationHelpers.py index ac16ea9c9f..7107d4df6a 100644 --- a/cura/OAuth2/AuthorizationHelpers.py +++ b/cura/OAuth2/AuthorizationHelpers.py @@ -51,7 +51,8 @@ class AuthorizationHelpers: self._token_url, data = urllib.parse.urlencode(data).encode("UTF-8"), headers_dict = headers, - callback = lambda response: self.parseTokenResponse(response, callback) + callback = lambda response: self.parseTokenResponse(response, callback), + error_callback = lambda response, _: self.parseTokenResponse(response, callback) ) def getAccessTokenUsingRefreshToken(self, refresh_token: str, callback: Callable[[AuthenticationResponse], None]) -> None: @@ -74,7 +75,7 @@ class AuthorizationHelpers: data = urllib.parse.urlencode(data).encode("UTF-8"), headers_dict = headers, callback = lambda response: self.parseTokenResponse(response, callback), - error_callback = lambda response: self.parseTokenResponse(response, callback) + error_callback = lambda response, _: self.parseTokenResponse(response, callback) ) def parseTokenResponse(self, token_response: QNetworkReply, callback: Callable[[AuthenticationResponse], None]) -> None: