chore(deps): bump hiero-ledger/hiero-solo-action from 0.19.0 to 0.21.0 (hiero-ledger#2322)

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
Signed-off-by: Manish Dait <daitmanish88@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: MontyPokemon <59332150+MonaaEid@users.noreply.github.com>
Co-authored-by: Manish Dait <daitmanish88@gmail.com>
Co-authored-by: Manish Dait <90558243+manishdait@users.noreply.github.com>

Author: 3 people

.github/workflows/pr-check-secondary-examples.yml

@@ -72,10 +72,10 @@ jobs:
 
       - name: Prepare Hiero Solo
         id: solo
-        uses: hiero-ledger/hiero-solo-action@328bc84c3b00a990a151418144fd682a4eb76ea6 #v0.19.0
+        uses: hiero-ledger/hiero-solo-action@c397dc8f75a4d24ae0be621529d7f9dfe8a9f460 # v0.21.0
         with:
           installMirrorNode: true
-          soloVersion: v0.69.0
+          soloVersion: v0.74.0
           mirrorNodeVersion: v0.153.0
           hieroVersion: v0.73.0
 

.github/workflows/pr-check-secondary-unit-integration-test.yml

@@ -129,10 +129,10 @@ jobs:
 
       - name: Prepare Hiero Solo
         id: solo
-        uses: hiero-ledger/hiero-solo-action@328bc84c3b00a990a151418144fd682a4eb76ea6 # v0.19.0
+        uses: hiero-ledger/hiero-solo-action@c397dc8f75a4d24ae0be621529d7f9dfe8a9f460 # v0.21.0
         with:
           installMirrorNode: true
-          soloVersion: v0.69.0
+          soloVersion: v0.74.0
           mirrorNodeVersion: v0.153.0
           hieroVersion: v0.73.0
 

src/hiero_sdk_python/address_book/registered_node_address_book_query.py

@@ -144,11 +144,11 @@ def execute(self, client) -> RegisteredNodeAddressBook:
 
     def _build_base_url(self, client) -> str:
         """Derive the mirror-node base URL (without ``/api/v1``)."""
-        rest_url = client.network.get_mirror_rest_url()  # e.g. http://localhost:5551/api/v1
+        rest_url = client.network.get_mirror_rest_url()  # e.g. http://localhost:38081/api/v1
 
         # For localhost / solo, use port 8084 for registered-node calls
-        if "localhost:5551" in rest_url or "127.0.0.1:5551" in rest_url:
-            rest_url = rest_url.replace(":5551", ":8084")
+        if "localhost:38081" in rest_url or "127.0.0.1:38081" in rest_url:
+            rest_url = rest_url.replace(":38081", ":8084")
 
         # Strip /api/v1 suffix so we can append full paths from pagination links
         return rest_url.removesuffix("/api/v1")

src/hiero_sdk_python/client/network.py

@@ -31,7 +31,7 @@ class Network:
         "mainnet": "https://mainnet-public.mirrornode.hedera.com",
         "testnet": "https://testnet.mirrornode.hedera.com",
         "previewnet": "https://previewnet.mirrornode.hedera.com",
-        "solo": "http://localhost:5551",  # Local development only
+        "solo": "http://localhost:38081",  # Local development only
     }
 
     DEFAULT_NODES: dict[str, list[_Node]] = {
@@ -61,9 +61,9 @@ class Network:
             ("2.previewnet.hedera.com:50211", AccountId(0, 0, 5)),
             ("3.previewnet.hedera.com:50211", AccountId(0, 0, 6)),
         ],
-        "solo": [("localhost:50211", AccountId(0, 0, 3))],
-        "localhost": [("localhost:50211", AccountId(0, 0, 3))],
-        "local": [("localhost:50211", AccountId(0, 0, 3))],
+        "solo": [("localhost:35211", AccountId(0, 0, 3))],
+        "localhost": [("localhost:35211", AccountId(0, 0, 3))],
+        "local": [("localhost:35211", AccountId(0, 0, 3))],
     }
 
     LEDGER_ID: dict[str, bytes] = {

src/hiero_sdk_python/query/fee_estimate_query.py

@@ -131,8 +131,8 @@ def execute(self, client) -> FeeEstimateResponse:
         mode = self._mode or FeeEstimateMode.INTRINSIC
         url = self._build_url(client, mode)
 
-        if "localhost:5551" in url or "127.0.0.1:5551" in url:
-            url = url.replace(":5551", ":8084")
+        if "localhost:38081" in url or "127.0.0.1:38081" in url:
+            url = url.replace(":38081", ":8084")
 
         self._ensure_frozen(self._transaction, client)
 

tests/integration/tls_integration_test.py

@@ -70,55 +70,6 @@ def test_tls_enabled_by_default_for_mainnet():
         client.close()
 
 
-@pytest.mark.integration
-def test_tls_disabled_by_default_for_localhost():
-    """Test that TLS is disabled by default for localhost network."""
-    network = Network("localhost")
-    client = Client(network)
-
-    try:
-        # Verify TLS is disabled by default
-        assert client.is_transport_security() is False, "TLS should be disabled by default for localhost"
-
-        # Verify certificate verification is still enabled by default
-        assert client.is_verify_certificates() is True, "Certificate verification should be enabled by default"
-
-        # Verify all nodes use plaintext ports (50211)
-        for node in network.nodes:
-            assert node._address._is_transport_security() is False, f"Node {node._account_id} should use plaintext port"
-            assert node._address._get_port() == 50211, f"Node {node._account_id} should use port 50211 for plaintext"
-    finally:
-        client.close()
-
-
-@pytest.mark.integration
-def test_tls_can_be_enabled_manually():
-    """Test that TLS can be enabled manually on networks where it's disabled by default."""
-    network = Network("localhost")
-    client = Client(network)
-
-    try:
-        # Initially TLS should be disabled
-        assert client.is_transport_security() is False, "TLS should be disabled by default for localhost"
-
-        # Enable TLS manually
-        client.set_transport_security(True)
-
-        # Verify TLS is now enabled
-        assert client.is_transport_security() is True, (
-            "TLS should be enabled after calling set_transport_security(True)"
-        )
-
-        # Verify all nodes now use TLS ports
-        for node in network.nodes:
-            assert node._address._is_transport_security() is True, (
-                f"Node {node._account_id} should use TLS port after enabling"
-            )
-            assert node._address._get_port() == 50212, f"Node {node._account_id} should use port 50212 for TLS"
-    finally:
-        client.close()
-
-
 @pytest.mark.integration
 def test_tls_can_be_disabled_manually():
     """Test that TLS can be disabled manually on networks where it's enabled by default."""

tests/unit/fee_estimate_query_test.py

@@ -331,3 +331,64 @@ def test_setters_are_chainable():
     result = q.set_max_attempts(2).set_max_backoff(1.0).set_high_volume_throttle(100)
 
     assert result is q
+
+
+def test_port_replacement_for_localhost_execute_single():
+    """Test localhost:38081 is replaced with :8084."""
+    client_1 = MagicMock()
+    client_1.network.get_mirror_rest_url.return_value = "http://localhost:38081/api/v1"
+
+    tx = (
+        TransferTransaction()
+        .add_hbar_transfer(AccountId.from_string("0.0.1001"), Hbar(-1))
+        .add_hbar_transfer(AccountId.from_string("0.0.1002"), Hbar(1))
+    )
+
+    query = FeeEstimateQuery().set_transaction(tx).set_mode(FeeEstimateMode.INTRINSIC)
+
+    with patch.object(query, "_execute_single") as mock_execute_single:
+        query.execute(client_1)
+        called_url = mock_execute_single.call_args[0][0]
+        assert ":8084" in called_url
+        assert ":38081" not in called_url
+
+    client_2 = MagicMock()
+    client_2.network.get_mirror_rest_url.return_value = "http://127.0.0.1:38081/api/v1"
+
+    with patch.object(query, "_execute_single") as mock_execute_single:
+        query.execute(client_2)
+        called_url = mock_execute_single.call_args[0][0]
+        assert ":8084" in called_url
+        assert ":38081" not in called_url
+
+
+def test_port_replacement_for_localhost_execute_multiple():
+    """Test localhost:38081 is replaced with :8084 for chunked tx."""
+    client_1 = MagicMock()
+    client_1.network.get_mirror_rest_url.return_value = "http://localhost:38081/api/v1"
+
+    tx = (
+        TopicMessageSubmitTransaction()
+        .set_topic_id(TopicId(0, 0, 4))
+        .set_message("A" * 2048)
+        .set_transaction_id(TransactionId.generate(AccountId(0, 0, 3)))
+        .set_node_account_id(AccountId(0, 0, 4))
+    )
+
+    query = FeeEstimateQuery().set_transaction(tx).set_mode(FeeEstimateMode.INTRINSIC)
+
+    with patch.object(query, "_execute_chunked", return_value=MagicMock()) as mock_execute_chunked:
+        query.execute(client_1)
+
+        called_url = mock_execute_chunked.call_args[0][1]
+        assert ":8084" in called_url
+        assert ":38081" not in called_url
+
+    client_2 = MagicMock()
+    client_2.network.get_mirror_rest_url.return_value = "http://127.0.0.1:38081/api/v1"
+
+    with patch.object(query, "_execute_chunked", return_value=MagicMock()) as mock_execute_chunked:
+        query.execute(client_2)
+        called_url = mock_execute_chunked.call_args[0][1]
+        assert ":8084" in called_url
+        assert ":38081" not in called_url

tests/unit/network_tls_test.py

@@ -43,19 +43,6 @@ def test_network_verification_enabled_by_default():
         assert network.is_verify_certificates() is True, f"Verification should be enabled for {network_name}"
 
 
-def test_network_set_transport_security_enable():
-    """Test enabling TLS on network."""
-    network = Network("solo")  # Starts with TLS disabled
-    assert network.is_transport_security() is False
-
-    network.set_transport_security(True)
-    assert network.is_transport_security() is True
-
-    # Verify all nodes are updated
-    for node in network.nodes:
-        assert node._address._is_transport_security() is True
-
-
 def test_network_set_transport_security_disable():
     """Test disabling TLS on network."""
     network = Network("testnet")  # Starts with TLS enabled

tests/unit/registered_node_model_test.py

@@ -512,16 +512,16 @@ def test_build_base_url_strips_api_v1(self):
         assert q._build_base_url(client) == "http://mirror.example.com"
 
     def test_build_base_url_localhost_port_replacement(self):
-        """Verify localhost:5551 is replaced with :8084."""
+        """Verify localhost:38081 is replaced with :8084."""
         q = RegisteredNodeAddressBookQuery()
-        client = self._make_client("http://localhost:5551/api/v1")
+        client = self._make_client("http://localhost:38081/api/v1")
         assert ":8084" in q._build_base_url(client)
-        assert ":5551" not in q._build_base_url(client)
+        assert ":38081" not in q._build_base_url(client)
 
     def test_build_base_url_127_port_replacement(self):
-        """Verify 127.0.0.1:5551 is replaced with :8084."""
+        """Verify 127.0.0.1:38081 is replaced with :8084."""
         q = RegisteredNodeAddressBookQuery()
-        client = self._make_client("http://127.0.0.1:5551/api/v1")
+        client = self._make_client("http://127.0.0.1:38081/api/v1")
         assert ":8084" in q._build_base_url(client)
 
     @patch("hiero_sdk_python.address_book.registered_node_address_book_query.requests.get")