bugfix: Improve the BlockCommitterHttpApi client to use url apis better

CHANGELOG.md

@@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Added
 - [2551](https://github.com/FuelLabs/fuel-core/pull/2551): Enhanced the DA compressed block header to include block id.
 
+### Fixed
+- [2599](https://github.com/FuelLabs/fuel-core/pull/2599): Use the proper `url` apis to construct full url path in `BlockCommitterHttpApi` client
+
 ## [Version 0.41.0]
 
 ### Added

crates/services/gas_price_service/src/v1/da_source_service/block_committer_costs.rs

@@ -128,11 +128,12 @@ impl BlockCommitterApi for BlockCommitterHttpApi {
         &self,
         l2_block_number: u32,
     ) -> DaBlockCostsResult<Vec<RawDaBlockCosts>> {
-        // Specific: http://localhost:8080/v1/costs?variant=specific&value=19098935&limit=5
+        // Specific: http://committer.url/v1/costs?variant=specific&value=19098935&limit=5
         if let Some(url) = &self.url {
             tracing::debug!("getting da costs by l2 block number: {l2_block_number}");
-            let formatted_url = format!("{url}/v1/costs?variant=specific&value={l2_block_number}&limit={NUMBER_OF_BUNDLES}");
-            let response = self.client.get(formatted_url).send().await?;
+            let path = format!("/v1/costs?variant=specific&value={l2_block_number}&limit={NUMBER_OF_BUNDLES}");
+            let full_path = url.join(&path)?;
+            let response = self.client.get(full_path).send().await?;
             let parsed = response.json::<Vec<RawDaBlockCosts>>().await?;
             Ok(parsed)
         } else {

crates/services/shared-sequencer/Cargo.toml

@@ -28,6 +28,7 @@ serde_json = "1.0"
 tendermint-rpc = { version = "0.36", features = ["http-client"] }
 tokio = { workspace = true }
 tracing = { workspace = true }
+url = { workspace = true }
 
 [features]
 test-helpers = []

crates/services/shared-sequencer/src/http_api.rs

@@ -1,6 +1,7 @@
 use anyhow::Context;
 use base64::prelude::*;
 use cosmrs::AccountId;
+use url::Url;
 
 mod api_types {
     use serde::Deserialize;
@@ -87,8 +88,10 @@ pub async fn estimate_transaction(
     let request = SimulateRequest {
         tx_bytes: tx_bytes.to_string(),
     };
+    let path = "/cosmos/tx/v1beta1/simulate";
+    let full_url = Url::parse(api_url)?.join(path).unwrap();
     let r = reqwest::Client::new()
-        .post(format!("{api_url}/cosmos/tx/v1beta1/simulate"))
+        .post(full_url)
         .json(&request)
         .send()
         .await?;