From 067858d445ed4685bff2bd07f3c23b2a0cd7e02e Mon Sep 17 00:00:00 2001 From: Corey Goldberg <1113081+cgoldberg@users.noreply.github.com> Date: Mon, 23 Jun 2025 12:31:33 -0400 Subject: [PATCH 1/2] [py] Return HTTP response reason when remote connection error occurs --- py/selenium/webdriver/remote/remote_connection.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/py/selenium/webdriver/remote/remote_connection.py b/py/selenium/webdriver/remote/remote_connection.py index 7ed47b56275b3..8a33fc4d9be0a 100644 --- a/py/selenium/webdriver/remote/remote_connection.py +++ b/py/selenium/webdriver/remote/remote_connection.py @@ -437,10 +437,10 @@ def _request(self, method, url, body=None): try: if 300 <= statuscode < 304: return self._request("GET", response.headers.get("location", None)) - if 399 < statuscode <= 500: - if statuscode == 401: - return {"status": statuscode, "value": "Authorization Required"} - return {"status": statuscode, "value": str(statuscode) if not data else data.strip()} + if statuscode == 401: + return {"status": statuscode, "value": "Authorization Required"} + if statuscode >= 400: + return {"status": statuscode, "value": f"{response.reason}" if not data else data.strip()} content_type = [] if response.headers.get("Content-Type", None): content_type = response.headers.get("Content-Type", None).split(";") From 96a755aa9f153d66678bdb4a34158abf03e305bf Mon Sep 17 00:00:00 2001 From: Corey Goldberg <1113081+cgoldberg@users.noreply.github.com> Date: Mon, 23 Jun 2025 12:39:52 -0400 Subject: [PATCH 2/2] [py] Remove f-string --- py/selenium/webdriver/remote/remote_connection.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/py/selenium/webdriver/remote/remote_connection.py b/py/selenium/webdriver/remote/remote_connection.py index 8a33fc4d9be0a..59b278b2b06a8 100644 --- a/py/selenium/webdriver/remote/remote_connection.py +++ b/py/selenium/webdriver/remote/remote_connection.py @@ -440,7 +440,7 @@ def _request(self, method, url, body=None): if statuscode == 401: return {"status": statuscode, "value": "Authorization Required"} if statuscode >= 400: - return {"status": statuscode, "value": f"{response.reason}" if not data else data.strip()} + return {"status": statuscode, "value": response.reason if not data else data.strip()} content_type = [] if response.headers.get("Content-Type", None): content_type = response.headers.get("Content-Type", None).split(";")