avoid exposing Headers

Author: dignifiedquire

src/headers/mod.rs

@@ -28,7 +28,7 @@ pub use to_header_values::ToHeaderValues;
 pub use values::Values;
 
 /// A collection of HTTP Headers.
-#[derive(Debug, Clone)]
+#[derive(Debug)]
 pub struct Headers {
     headers: HashMap<HeaderName, Vec<HeaderValue>>,
 }
@@ -144,6 +144,25 @@ impl<'a> IntoIterator for &'a mut Headers {
     }
 }
 
+impl std::iter::FromIterator<(HeaderName, Vec<HeaderValue>)> for Headers {
+    fn from_iter<I: IntoIterator<Item = (HeaderName, Vec<HeaderValue>)>>(iter: I) -> Self {
+        Headers {
+            headers: iter.into_iter().collect(),
+        }
+    }
+}
+
+impl<'a> std::iter::FromIterator<(&'a HeaderName, &'a Vec<HeaderValue>)> for Headers {
+    fn from_iter<I: IntoIterator<Item = (&'a HeaderName, &'a Vec<HeaderValue>)>>(iter: I) -> Self {
+        Headers {
+            headers: iter
+                .into_iter()
+                .map(|(name, values)| (name.clone(), values.clone()))
+                .collect(),
+        }
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

src/request.rs

@@ -55,8 +55,11 @@ impl Request {
     }
 
     /// Set the headers.
-    pub fn set_headers(&mut self, headers: Headers) {
-        self.headers = headers;
+    pub fn set_headers<'a, T: IntoIterator<Item = (&'a HeaderName, &'a Vec<HeaderValue>)>>(
+        &mut self,
+        headers: T,
+    ) {
+        self.headers = headers.into_iter().collect();
     }
 
     /// Get a reference to the url.