Pass options to HTTPBuilder in parse_url_opts (#5310) (#5311)

apache · Jan 19, 2024 · ff5a84d · ff5a84d · CarlKCarlK · Jan 19, 2024
1 parent 8301b23
commit ff5a84d
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/object_store/src/parse.rs b/object_store/src/parse.rs
@@ -105,7 +105,7 @@ impl ObjectStoreScheme {
     }
 }
 
-#[cfg(any(feature = "aws", feature = "gcp", feature = "azure"))]
+#[cfg(feature = "cloud")]
 macro_rules! builder_opts {
     ($builder:ty, $url:expr, $options:expr) => {{
         let builder = $options.into_iter().fold(
@@ -164,8 +164,7 @@ where
         }
         #[cfg(feature = "http")]
         ObjectStoreScheme::Http => {
-            let url = &url[..url::Position::BeforePath];
-            Box::new(crate::http::HttpBuilder::new().with_url(url).build()?) as _
+            builder_opts!(crate::http::HttpBuilder, url, _options)
         }
         #[cfg(not(all(feature = "aws", feature = "azure", feature = "gcp", feature = "http")))]
         s => {