HttpStatusError: store status code

It can be convenient to access after the fact instead of passing the
msg.

Closes pyodide#4973

Author: Carreau

docs/project/changelog.md

@@ -21,6 +21,9 @@ myst:
   timezone handling.
   {pr}`4889`
 
+- {{ Enhancement }} `HttpStatusError` now store their the corresponding request
+  `status`, `status_message` and `url` {pr}`4974`.
+
 - {{ Enhancement }} Added implementation to abort `pyfetch` and `FetchResponse`
   manually or automatically.
   {pr}`4846`

src/py/pyodide/http.py

@@ -28,7 +28,14 @@
 
 
 class HttpStatusError(OSError):
+    status: int
+    status_text: str
+    url: str
+
     def __init__(self, status: int, status_text: str, url: str) -> None:
+        self.status = status
+        self.status_text = status_text
+        self.url = url
         if 400 <= status < 500:
             super().__init__(f"{status} Client Error: {status_text} for url: {url}")
         elif 500 <= status < 600:

src/tests/test_pyodide_http.py

@@ -80,27 +80,38 @@ def raise_for_status_fixture(httpserver):
 
 
 @run_in_pyodide
-async def test_pyfetch_raise_for_status(selenium, raise_for_status_fixture):
+async def test_pyfetch_raise_for_status_does_not_raise_200(
+    selenium, raise_for_status_fixture
+):
     import pytest
 
-    from pyodide.http import pyfetch
+    from pyodide.http import HttpStatusError, pyfetch
 
     resp = await pyfetch(raise_for_status_fixture["/status_200"])
     resp.raise_for_status()
     assert await resp.string() == "Some data here!"
 
     resp = await pyfetch(raise_for_status_fixture["/status_404"])
     with pytest.raises(
-        OSError, match="404 Client Error: NOT FOUND for url: .*/status_404"
-    ):
+        HttpStatusError, match="404 Client Error: NOT FOUND for url: .*/status_404"
+    ) as error_404:
         resp.raise_for_status()
 
+    assert error_404.value.status == 404
+    assert error_404.value.status_text == "NOT FOUND"
+    assert error_404.value.url.endswith("status_404")
+
     resp = await pyfetch(raise_for_status_fixture["/status_504"])
     with pytest.raises(
-        OSError, match="504 Server Error: GATEWAY TIMEOUT for url: .*/status_504"
-    ):
+        HttpStatusError,
+        match="504 Server Error: GATEWAY TIMEOUT for url: .*/status_504",
+    ) as error_504:
         resp.raise_for_status()
 
+    assert error_504.value.status == 504
+    assert error_504.value.status_text == "GATEWAY TIMEOUT"
+    assert error_504.value.url.endswith("status_504")
+
 
 @run_in_pyodide
 async def test_pyfetch_unpack_archive(selenium):