Invalid transformation to http::Response<Vec<u8>> for non utf8 response bodies

[rogercoll]

Invalid transformation to http::Response> for non utf8 response bodies

from ureq.

[jsha]

Thanks for reporting this. Looking at that code, it's just wrong regardless of content type.

Instead of checking content type, that code should unconditionally call into_reader(), then read the result into a Vec<u8>.

[rogercoll]

Make sense to me. Wdyt of #702?

[rogercoll]

It would be great to have the happy path transformation too (TryFrom). Which would be the best way to define the possible error? (E.g io::Error)

Would it be possible to extend the ureq::Error with a new enum variant:

#[derive(Debug)]
pub enum Error {
    /// A response was successfully received but had status code >= 400.
    /// Values are (status_code, Response).
    Status(u16, Response),
    /// There was an error making the request or receiving the response.
    Transport(Transport),
     #[cfg(features = ...)]
     /// There was an error while transforming the request to http::Request.
    ConvertError(String),
}