-
Notifications
You must be signed in to change notification settings - Fork 347
IP is not recognized in RTSP url #606
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
The URL Standard has special handling for several URL schemes (one of which is |
It's not really important . It's just inconvenient. I have to write code like this: match host {
Host::Domain(name) => {
if let Ok(ip) = name.parse::<IpAddr>() {
ip
} else {
resolve(name)
}
}
Host::Ipv4(ip) => IpAddr::V4(ip),
Host::Ipv6(ip) => IpAddr::V6(ip),
} to gen ip address from url. I didn't know that this was the intended behavior, it looked like a bug to me. Since that's how the URL Standard requires the parser to work the issue could be closed. |
What is the motivation of that? I can't think of any scheme, except maybe "file", where the host can't be an ip address. |
See here: #577 (comment). |
I would expect the following assert to pass:
but it does not:
The text was updated successfully, but these errors were encountered: