to_file_path() on localhost returns Path::new("/") about rust-url HOT 4 CLOSED

I agree that in this case it's confusing but I'm confused about what you expect it to print out? Frankly I would expect an example with file:///home/alexcrichton/path/to/some.txt to be a more useful test of this functionality.

from rust-url.

to_file_path doesn’t have a type parameter anymore, but other than that this behavior hasn’t changed. Still, it matches the doc-comment. What outcome would you rather have?

Assuming the URL is in the file scheme or similar, convert its path to an absolute std::path::Path.

Note: This does not actually check the URL’s scheme, and may give nonsensical results for other schemes. It is the user’s responsibility to check the URL’s scheme before calling this.

# use url::Url;
# let url = Url::parse("file:///etc/passwd").unwrap();
let path = url.to_file_path();

Returns Err if the host is neither empty nor "localhost", or if Path::new_opt() returns None. (That is, if the percent-decoded path contains a NUL byte or, for a Windows path, is not UTF-8.)

from rust-url.

Based on the function's documentation, it seems like this is behaving correctly. What action is this issue tracking?

from rust-url.

Closing as working as intended / document. Feel free to reopen or comment.

from rust-url.