Troubled Proxy Parsing?

[Coding-Badly]

Hello Sean!

I believe this error handler is troubled.

I expected the call to reqwest::Proxy::http to return Err(InvalidIpv4Address) as the call to reqwest::Url::parse does.

fn main() -> Result<(), Box<dyn std::error::Error>> {
    println!();

    const BAD_NEWS: &str = "http://257.317.406.832";

    println!("{:?}", reqwest::Url::parse(BAD_NEWS));
    // Err(InvalidIpv4Address)

    println!("{:?}", reqwest::Proxy::http(BAD_NEWS));
    // Ok(Proxy(Http(http://http), None))

    println!();
    Ok(())
}

I think a not - trim_start - starts_with for http:// and https:// will solve the problem.

But, I believe the ideal solution is to remove the automatic error handling. In my mind adjusting the URL belongs exclusively to the library user.