Sync reader with Seek+Read implementation

.github/workflows/test.yml

@@ -11,7 +11,7 @@ jobs:
           - wasm32-unknown-unknown
 
     name: Build Check ${{ matrix.target }}
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
 
     steps:
       - uses: actions/checkout@v2
@@ -28,7 +28,7 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: check
-          args: --target ${{matrix.target}} --all-features
+          args: --target ${{matrix.target}} --no-default-features --features=reqwest-async
 
   test:
     strategy:
@@ -37,7 +37,7 @@ jobs:
           - x86_64-unknown-linux-gnu
 
     name: Test ${{ matrix.target }}
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-22.04
 
     steps:
       - uses: actions/checkout@v2
@@ -60,14 +60,10 @@ jobs:
         uses: actions-rs/cargo@v1
         with:
           command: test
-          args: --target ${{matrix.target}} --all-features
-
-      # This is currently broken because
-      # reqwest is not actually optional
-      #
-      # - name: test --no-default-features
-      #   uses: actions-rs/cargo@v1
-      #   with:
-      #     command: test
-      #     args: --target ${{matrix.target}} --no-default-features
+          args: --target ${{matrix.target}} --all-features -- --test-threads 1
 
+      - name: test --no-default-features
+        uses: actions-rs/cargo@v1
+        with:
+          command: test
+          args: --target ${{matrix.target}} --no-default-features

Cargo.toml

@@ -11,15 +11,21 @@ license = "MIT/Apache-2.0"
 keywords = ["http", "reader", "buffer"]
 
 [features]
-default = ["reqwest"]
+default = ["reqwest-sync", "reqwest-async"]
+reqwest-async = ["reqwest"]
+reqwest-sync = ["reqwest/blocking"]
 
 [dependencies]
+async-trait = "0.1.51"
 byteorder = "1.4.2"
+bytes = "1.0.1"
+log = "0.4.13"
 reqwest = { version = "0.11.0", default-features = false, features = ["default-tls"], optional = true }
-bytes = { version = "1.0.1" }
 thiserror = "1.0"
-log = "0.4.13"
-async-trait = "0.1.51"
 
 [dev-dependencies]
+env_logger = "0.10.0"
 tokio = { version = "1.0.2", default-features = false, features = ["rt-multi-thread", "macros"] }
+
+[package.metadata.docs.rs]
+all-features = true

README.md

@@ -2,11 +2,21 @@
 
 HTTP client for HTTP Range requests with a buffer optimized for sequential requests.
 
+Implements Seek+Read for blocking clients, which makes it a drop-in replacement for local files.
 
-Usage example:
+Usage examples:
 
     use http_range_client::*;
 
     let mut client = BufferedHttpRangeClient::new("https://flatgeobuf.org/test/data/countries.fgb");
-    let bytes = client.get_range(0, 3, 256).await?;
-    assert_eq!(bytes, "fgb".as_bytes());
+    let bytes = client.min_req_size(256).get_range(0, 3).await?;
+    assert_eq!(bytes, b"fgb");
+    let version = client.get_bytes(1).await?; // From buffer - no HTTP request!
+    assert_eq!(version, [3]);
+
+    let mut client =
+        HttpReader::new("https://www.rust-lang.org/static/images/favicon-32x32.png");
+    client.seek(SeekFrom::Start(1)).ok();
+    let mut bytes = [0; 3];
+    client.read_exact(&mut bytes)?;
+    assert_eq!(&bytes, b"PNG");