feat(client): add conn::Builder::max_buf_size()

This allows users to configure a limit to client connections' read and
write buffers.

Closes hyperium#1748

src/client/conn.rs

@@ -75,6 +75,7 @@ pub struct Builder {
     h1_writev: bool,
     h1_title_case_headers: bool,
     h1_read_buf_exact_size: Option<usize>,
+    h1_max_buf_size: Option<usize>,
     http2: bool,
 }
 
@@ -435,6 +436,7 @@ impl Builder {
             h1_writev: true,
             h1_read_buf_exact_size: None,
             h1_title_case_headers: false,
+            h1_max_buf_size: None,
             http2: false,
         }
     }
@@ -462,6 +464,17 @@ impl Builder {
         self.h1_read_buf_exact_size = sz;
         self
     }
+
+    pub(super) fn h1_max_buf_size(&mut self, max: usize) -> &mut Self {
+        assert!(
+            max >= proto::h1::MINIMUM_MAX_BUFFER_SIZE,
+            "the max_buf_size cannot be smaller than the minimum that h1 specifies."
+        );
+
+        self.h1_max_buf_size = Some(max);
+        self
+    }
+
     /// Sets whether HTTP2 is required.
     ///
     /// Default is false.
@@ -510,6 +523,9 @@ where
             if let Some(sz) = self.builder.h1_read_buf_exact_size {
                 conn.set_read_buf_exact_size(sz);
             }
+            if let Some(max) = self.builder.h1_max_buf_size {
+                conn.set_max_buf_size(max);
+            }
             let cd = proto::h1::dispatch::Client::new(rx);
             let dispatch = proto::h1::Dispatcher::new(cd, conn);
             Either::A(dispatch)

src/client/mod.rs

@@ -901,13 +901,31 @@ impl Builder {
 
     /// Sets the exact size of the read buffer to *always* use.
     ///
+    /// Note that this setting conflicts with the `http1_max_buf_size` option.
+    ///
     /// Default is an adaptive read buffer.
     #[inline]
     pub fn http1_read_buf_exact_size(&mut self, sz: usize) -> &mut Self {
         self.conn_builder.h1_read_buf_exact_size(Some(sz));
         self
     }
 
+    /// Set the maximum buffer size for the connection.
+    ///
+    /// Default is ~400kb.
+    ///
+    /// Note that this setting does not affect HTTP/2.
+    /// Also note that this setting conflicts with the `http1_read_buf_exact_size` option.
+    ///
+    /// # Panics
+    ///
+    /// The minimum value allowed is 8192. This method panics if the passed `max` is less than the minimum.
+    #[inline]
+    pub fn http1_max_buf_size(&mut self, max: usize) -> &mut Self {
+        self.conn_builder.h1_max_buf_size(max);
+        self
+    }
+
     /// Set whether HTTP/1 connections will write header names as title case at
     /// the socket level.
     ///