Merge branch 'algesten:main' into main

src/chunked/mod.rs

@@ -12,7 +12,7 @@
 // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 // See the License for the specific language governing permissions and
 // limitations under the License.
-#![allow(dead_code)]
+#![allow(dead_code, unused_imports)]
 mod decoder;
 pub use decoder::Decoder;
 

src/error.rs

@@ -212,7 +212,7 @@ impl Display for Error {
         match self {
             Error::Status(status, response) => {
                 write!(f, "{}: status code {}", response.get_url(), status)?;
-                if let Some(original) = response.history.get(0) {
+                if let Some(original) = response.history.first() {
                     write!(f, " (redirected from {})", original)?;
                 }
             }

src/header.rs

@@ -115,7 +115,7 @@ impl Header {
     ///
     /// ureq can't know what encoding the header is in, but this function provides
     /// an escape hatch for users that need to handle such headers.
-    #[cfg(any(feature = "http-interop", feature = "charset"))]
+    #[allow(unused)]
     pub fn value_raw(&self) -> &[u8] {
         let mut bytes = &self.line.as_bytes()[self.index + 1..];
 
@@ -159,7 +159,7 @@ pub fn get_header<'h>(headers: &'h [Header], name: &str) -> Option<&'h str> {
         .and_then(|h| h.value())
 }
 
-#[cfg(any(doc, all(test, any(feature = "http-interop", feature = "http-crate"))))]
+#[allow(unused)]
 pub fn get_header_raw<'h>(headers: &'h [Header], name: &str) -> Option<&'h [u8]> {
     headers
         .iter()

src/http_crate.rs

@@ -8,9 +8,9 @@ use crate::{header::HeaderLine, response::ResponseStatusIndex, Request, Response
 /// Converts an [`http::Response`] into a [`Response`].
 ///
 /// As an [`http::Response`] does not contain a URL, `"https://example.com/"` is used as a
-/// placeholder. Additionally, if the response has a header which cannot be converted into a valid
-/// [`Header`](crate::Header), it will be skipped rather than having the conversion fail. The remote
-/// address property will also always be `127.0.0.1:80` for similar reasons to the URL.
+/// placeholder. Additionally, if the response has a header which cannot be converted to ureq's
+/// internal header representation, it will be skipped rather than having the conversion fail.
+/// The remote address property will also always be `127.0.0.1:80` for similar reasons to the URL.
 ///
 /// ```
 /// # fn main() -> Result<(), http::Error> {