add better error handling for invalid client URLs

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [[Unreleased]]
 ### Added:
 - Support for dynamic fee calculation
+- Better error handling for invalid client URL
 
 ### Fixed
 - Resolve `txnNotFound` error with `send_reliable_submission` when waiting for a submitted malformed transaction

tests/unit/clients/test_json_rpc_client.py

@@ -0,0 +1,26 @@
+"""Test the json_rpc_client."""
+from __future__ import annotations
+
+from unittest import TestCase
+from xrpl.asyncio.clients.exceptions import XRPLRequestFailureException
+
+from xrpl.clients import JsonRpcClient
+from xrpl.models.requests import ServerInfo
+
+
+class TestJsonRpcClient(TestCase):
+    """Test json_rpc_client."""
+
+    def test_json_rpc_client_valid_url(self: TestJsonRpcClient) -> None:
+        # Valid URL
+        JSON_RPC_URL = "https://s.altnet.rippletest.net:51234/"
+        client = JsonRpcClient(JSON_RPC_URL)
+        client.request(ServerInfo())
+
+    def test_json_rpc_client_invalid_url(self: TestJsonRpcClient) -> None:
+        # Invalid URL
+        JSON_RPC_URL = "https://s2.ripple.com:51233/"
+        with self.assertRaises(XRPLRequestFailureException):
+            client = JsonRpcClient(JSON_RPC_URL)
+            client.request(ServerInfo())
+

xrpl/asyncio/clients/json_rpc_base.py

@@ -5,6 +5,7 @@
 from typing_extensions import Final
 
 from xrpl.asyncio.clients.client import Client
+from xrpl.asyncio.clients.exceptions import XRPLRequestFailureException
 from xrpl.asyncio.clients.utils import json_to_response, request_to_json_rpc
 from xrpl.models.requests.request import Request
 from xrpl.models.response import Response
@@ -36,4 +37,9 @@ async def request_impl(self: JsonRpcBase, request: Request) -> Response:
                 self.url,
                 json=request_to_json_rpc(request),
             )
+            if response.status_code != 200:
+                raise XRPLRequestFailureException({
+                    'error': response.status_code,
+                    'error_message': response.text,
+                })
             return json_to_response(response.json())