Replace reqwest with ureq (#1407)

* Replace reqwest with ureq for less dependencies * Simplify code * Cleanup
rerun-io · Mar 2, 2023 · 802923c · 802923c
1 parent c8502d7
commit 802923c
Show file tree

Hide file tree

Showing 9 changed files with 237 additions and 83 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -69,7 +69,6 @@ polars-core = "0.27.1"
 polars-lazy = "0.27.1"
 polars-ops = "0.27.1"
 puffin = "0.14"
-reqwest = { version = "0.11", default-features = false }    # TODO(emilk): consider replacing `reqwest` with `ureq`
 thiserror = "1.0"
 time = "0.3"
 tokio = "1.24"

diff --git a/crates/re_analytics/Cargo.toml b/crates/re_analytics/Cargo.toml
@@ -34,7 +34,9 @@ uuid = { version = "1.1", features = ["serde", "v4", "js"] }
 
 [target.'cfg(not(target_arch = "wasm32"))'.dependencies]
 directories-next = "2"
-reqwest = { workspace = true, features = ["blocking", "rustls-tls"] }
+ureq = { version = "2.6", features = [
+  "json",
+] } # TODO(emilk): use ehttp for web supprt
 
 [target.'cfg(target_arch = "wasm32")'.dependencies]
 web-sys = { version = "0.3.58", features = ["Storage"] }

diff --git a/crates/re_analytics/src/pipeline_native.rs b/crates/re_analytics/src/pipeline_native.rs
@@ -25,9 +25,6 @@ pub enum PipelineError {
 
     #[error(transparent)]
     Serde(#[from] serde_json::Error),
-
-    #[error(transparent)]
-    Http(#[from] reqwest::Error),
 }
 
 /// An eventual, at-least-once(-ish) event pipeline, backed by a write-ahead log on the local disk.