diff --git a/securedrop_client/logic.py b/securedrop_client/logic.py index b6dd34491f..9c9d7fbaf6 100644 --- a/securedrop_client/logic.py +++ b/securedrop_client/logic.py @@ -312,7 +312,7 @@ def login(self, username, password, totp): """ storage.mark_all_pending_drafts_as_failed(self.session) self.api = sdclientapi.API( - self.hostname, username, password, totp, self.proxy, default_request_timeout=60) + self.hostname, username, password, totp, self.proxy, default_request_timeout=20) self.call_api(self.api.authenticate, self.on_authenticate_success, self.on_authenticate_failure) diff --git a/securedrop_client/queue.py b/securedrop_client/queue.py index 934486e04e..58aec7f2aa 100644 --- a/securedrop_client/queue.py +++ b/securedrop_client/queue.py @@ -136,13 +136,12 @@ class ApiJobQueue(QObject): def __init__(self, api_client: API, session_maker: scoped_session) -> None: super().__init__(None) - self.api_client = api_client self.main_thread = QThread() self.download_file_thread = QThread() - self.main_queue = RunnableQueue(self.api_client, session_maker) - self.download_file_queue = RunnableQueue(self.api_client, session_maker) + self.main_queue = RunnableQueue(api_client, session_maker) + self.download_file_queue = RunnableQueue(api_client, session_maker) self.main_queue.moveToThread(self.main_thread) self.download_file_queue.moveToThread(self.download_file_thread) @@ -159,11 +158,6 @@ def logout(self) -> None: def login(self, api_client: API) -> None: logger.debug('Passing API token to queues') - - # Setting realistic (shorter) timeout for general requests so that user feedback - # is faster - api_client.default_request_timeout = 5 - self.main_queue.api_client = api_client self.download_file_queue.api_client = api_client self.start_queues()