Replace &HeaderName with Into<HeaderName>

Author: yoshuawuyts

src/headers/headers.rs

@@ -64,7 +64,7 @@ impl Headers {
         values: impl ToHeaderValues,
     ) -> crate::Result<()> {
         let name = name.into();
-        match self.get_mut(&name) {
+        match self.get_mut(name.clone()) {
             Some(headers) => {
                 let mut values: HeaderValues = values.to_header_values()?.collect();
                 headers.append(&mut values);
@@ -82,13 +82,13 @@ impl Headers {
     }
 
     /// Get a mutable reference to a header.
-    pub fn get_mut(&mut self, name: &HeaderName) -> Option<&mut HeaderValues> {
-        self.headers.get_mut(name)
+    pub fn get_mut(&mut self, name: impl Into<HeaderName>) -> Option<&mut HeaderValues> {
+        self.headers.get_mut(&name.into())
     }
 
     /// Remove a header.
-    pub fn remove(&mut self, name: &HeaderName) -> Option<HeaderValues> {
-        self.headers.remove(name)
+    pub fn remove(&mut self, name: impl Into<HeaderName>) -> Option<HeaderValues> {
+        self.headers.remove(&name.into())
     }
 
     /// An iterator visiting all header pairs in arbitrary order.

src/request.rs

@@ -369,13 +369,13 @@ impl Request {
     }
 
     /// Get a mutable reference to a header.
-    pub fn header_mut(&mut self, name: &HeaderName) -> Option<&mut HeaderValues> {
-        self.headers.get_mut(name)
+    pub fn header_mut(&mut self, name: impl Into<HeaderName>) -> Option<&mut HeaderValues> {
+        self.headers.get_mut(name.into())
     }
 
     /// Remove a header.
-    pub fn remove_header(&mut self, name: &HeaderName) -> Option<HeaderValues> {
-        self.headers.remove(name)
+    pub fn remove_header(&mut self, name: impl Into<HeaderName>) -> Option<HeaderValues> {
+        self.headers.remove(name.into())
     }
 
     /// Set an HTTP header.

src/response.rs

@@ -69,18 +69,18 @@ impl Response {
     }
 
     /// Get a mutable reference to a header.
-    pub fn header_mut(&mut self, name: &HeaderName) -> Option<&mut HeaderValues> {
-        self.headers.get_mut(name)
+    pub fn header_mut(&mut self, name: impl Into<HeaderName>) -> Option<&mut HeaderValues> {
+        self.headers.get_mut(name.into())
     }
 
     /// Get an HTTP header.
     pub fn header(&self, name: impl Into<HeaderName>) -> Option<&HeaderValues> {
-        self.headers.get(name)
+        self.headers.get(name.into())
     }
 
     /// Remove a header.
-    pub fn remove_header(&mut self, name: &HeaderName) -> Option<HeaderValues> {
-        self.headers.remove(name)
+    pub fn remove_header(&mut self, name: impl Into<HeaderName>) -> Option<HeaderValues> {
+        self.headers.remove(name.into())
     }
 
     /// Set an HTTP header.

src/security/mod.rs

@@ -105,7 +105,7 @@ pub fn powered_by(mut headers: impl AsMut<Headers>, value: Option<HeaderValue>)
             headers.as_mut().insert(name, value);
         }
         None => {
-            headers.as_mut().remove(&name);
+            headers.as_mut().remove(name);
         }
     };
 }

src/trailers.rs

@@ -121,17 +121,17 @@ impl Trailers {
     }
 
     /// Get a reference to a header.
-    pub fn get(&self, name: impl Into<HeaderName>) -> Option<&HeaderValues> {
+    pub fn get(&self, name: &impl Into<HeaderName>) -> Option<&HeaderValues> {
         self.headers.get(name)
     }
 
     /// Get a mutable reference to a header.
-    pub fn get_mut(&mut self, name: &HeaderName) -> Option<&mut HeaderValues> {
+    pub fn get_mut(&mut self, name: impl Into<HeaderName>) -> Option<&mut HeaderValues> {
         self.headers.get_mut(name)
     }
 
     /// Remove a header.
-    pub fn remove(&mut self, name: &HeaderName) -> Option<HeaderValues> {
+    pub fn remove(&mut self, name: impl Into<HeaderName>) -> Option<HeaderValues> {
         self.headers.remove(name)
     }