chore: removed redundant return statements from handle_health_status

Author: Voyz

ibind/client/ibkr_client.py

@@ -295,16 +295,12 @@ def handle_health_status(self) -> bool:
         except ExternalBrokerError as e:
             if "Failed to resolve 'api.ibkr.com'" in str(e):
                 _LOGGER.error('Connection to IBKR servers failed during reauthentication. Check internet connection between IBind and \'api.ibkr.com\'')
-                return False
             elif 'An attempt was made to access a socket in a way forbidden by its access permissions' in str(e):
                 _LOGGER.error('Connection to IBKR servers blocked during reauthentication. Check that nothing is blocking connectivity of the application')
-                return False
             elif e.status_code == 410 and 'gone' in str(e):
                 _LOGGER.error(f'OAuth 410 gone: recreate a new live session token, or try a different server, eg. "1.api.ibkr.com", "2.api.ibkr.com", etc.')
-                return False
             else:
                 _LOGGER.error(f'Unknown error checking IBKR connection during reauthentication: {exception_to_string(e)}')
-                return False
         except Exception as e: # pragma: no cover
             _LOGGER.error(f'Error reauthenticating OAuth during reauthentication: {exception_to_string(e)}')
         return False