Does not return ParseError for invalid Url

[Parth]

use url::Url;

fn main() {
    println!("{:?}", Url::parse("http://\""));
}

Outputs:

Ok("http://\"/")

[Parth]

Breaks Reqwests

[SimonSapin]

I haven’t looked in details, but it may be that the set of strings that successfully parse as a URL in the URL Standard (which this crate implements, though it’s behind on some spec changes) is not the same as the set of strings that match the request-target grammar in HTTP RFCs (which http::Uri implements).

[timvisee]

Here are few other ones that parse successfully, but are likely invalid (via seanmonstar/reqwest#668 (comment)):

"http://‘nvme@nip2"
"http://-1@"
"https://a@b"
"https://€@qq"

[timvisee]

I discovered that Url currently accepts any character (except for a few blacklisted ones) as part of a hostname. I believe this might be the root cause of this issue.

Opened this issue for it: #554