setting default auto_decode to true

Author: jaytaph

CHANGELOG.md

@@ -37,6 +37,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `RequestBody`'s `bytes` field is private; use `RequestBody::as_bytes()`.
   `len()` now returns `Option<u64>` (`None` for a stream without a declared
   length).
+- `FetchRequest::builder()` now defaults to `auto_decode: true`, matching the
+  simple API and the wasm32 build. Use `.with_auto_decode(false)` for raw bytes.
+- With decoding on, `max_bytes` caps the decompressed size, and the early
+  `Content-Length` rejection no longer applies (reqwest strips the header
+  when it decodes).
 
 ### Fixed
 

src/net/types.rs

@@ -458,7 +458,7 @@ impl FetchRequest {
 
 /// Builder for [`FetchRequest`], created via [`FetchRequest::builder`].
 ///
-/// All settings are optional; `build()` produces a buffered, non-decoding request
+/// All settings are optional; `build()` produces a buffered, decoding request
 /// with [`Priority::Normal`] unless configured otherwise.
 pub struct FetchRequestBuilder {
     reference: RequestReference,
@@ -488,7 +488,7 @@ impl FetchRequestBuilder {
             initiator: Initiator::default(),
             kind: ResourceKind::default(),
             streaming: false,
-            auto_decode: false,
+            auto_decode: true,
             max_bytes: None,
             body: None,
         }
@@ -530,7 +530,10 @@ impl FetchRequestBuilder {
         self
     }
 
-    /// Sets whether to transparently decode compressed responses (default: false)
+    /// Sets whether to transparently decode compressed responses (default: true).
+    ///
+    /// With decoding on, [`with_max_bytes`](Self::with_max_bytes) caps the decompressed
+    /// size; set `false` to cap bytes as they arrive on the wire.
     pub fn with_auto_decode(mut self, auto_decode: bool) -> Self {
         self.auto_decode = auto_decode;
         self
@@ -679,6 +682,13 @@ mod tests {
         assert_eq!(buffered.as_bytes().map(|b| b.len()), Some(3));
     }
 
+    #[test]
+    fn builder_decodes_by_default() {
+        let fr =
+            FetchRequest::builder(Method::GET, Url::parse("https://example.org").unwrap()).build();
+        assert!(fr.auto_decode);
+    }
+
     #[test]
     fn fetch_request_generate_get_and_headers() {
         let mut fr = FetchRequest::builder(