Increase visibility of error message when action fails due to HTTP request error

The "arduino/report-size-deltas" action relies on the GitHub REST API. It makes HTTP requests to various API endpoints
during each workflow run.

These requests might produce an exception due to transient network outages, misconfiguration of the workflow, or a
problem with the action. This will cause the action's step in the workflow run to fail. The maintainer of the repository
in which the action is used may then need to investigate the cause of the failure.

Previously, the cause of the failure could only be determined by evaluating the workflow run logs, which are somewhat
unfriendly due to containing the Python stack trace produced by the HTTP request exception. The significant error
message is hereby made visible as an annotation on the workflow run summary page through the use of an "error" workflow
command.

Author: per1234

reportsizedeltas/reportsizedeltas.py

@@ -630,19 +630,26 @@ def raw_http_request(self, url: str, data: bytes | None = None):
         request = urllib.request.Request(url=url, headers=headers, data=data)
 
         retry_count = 0
-        while retry_count <= maximum_urlopen_retries:
-            retry_count += 1
+        while True:
             try:
                 # The rate limit API is not subject to rate limiting
                 if url.startswith("https://api.github.com") and not url.startswith("https://api.github.com/rate_limit"):
                     self.handle_rate_limiting()
                 return urllib.request.urlopen(url=request)
             except urllib.error.HTTPError as exception:
-                if not determine_urlopen_retry(exception=exception):
-                    raise exception
+                if determine_urlopen_retry(exception=exception):
+                    if retry_count < maximum_urlopen_retries:
+                        retry_count += 1
+                        continue
+                    else:
+                        # Maximum retries reached without successfully opening URL
+                        print("Maximum number of URL load retries exceeded")
+
+                print(f"::error::{exception.__class__.__name__}: {exception}")
+                for line in exception.fp:
+                    print(line.decode(encoding="utf-8", errors="ignore"))
 
-        # Maximum retries reached without successfully opening URL
-        raise TimeoutError("Maximum number of URL load retries exceeded")
+                raise exception
 
     def handle_rate_limiting(self) -> None:
         """Check whether the GitHub API request limit has been reached.

reportsizedeltas/tests/test_reportsizedeltas.py

@@ -910,7 +910,7 @@ def test_raw_http_request(mocker):
 
     # urllib.request.urlopen() has potentially recoverable exceptions, but exceeds retry count
     reportsizedeltas.determine_urlopen_retry.return_value = True
-    with pytest.raises(expected_exception=TimeoutError):
+    with pytest.raises(expected_exception=urllib.error.HTTPError):
         report_size_deltas.raw_http_request(url=url, data=data)