Use try_from_bytes in handle_file_urls

[kazatsuyu]

Reduce build error on Windows.

Release Notes:

• N/A

[kazatsuyu]

Note: try_from_bytes always returns Ok on [cfg(unix)].

[ConradIrwin]

Because we don't control the input to this method, we should not panic if the bytes are invalid, but just ignore them. I think the following would be better:

Suggested change

	PathBuf::try_from_bytes(decoded.as_ref()).unwrap()
	.flat_map(|url| {
	let decoded = urlencoding::decode_binary(url.as_bytes());
	PathBuf::try_from_bytes(decoded.as_ref()).log_err()
	})

[kazatsuyu]

I fixed this

[SomeoneToIgnore]

I have no objections from merging that in if the reviewer finds it appropriate, but strongly urge you to consider adding Windows runners (GitHub seem to allow the standard ones: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#using-a-github-hosted-runner so we can do smth. like #7581).
Without the CI, there's no guarantees that this won't degrade again, making the PR pointless in the long run.

I would not expect the code to 100% compile under Windows now, but some of the crates may be already?
If not, an effort could be made to make certain crate(s) to compile and add the Windows CI to check those first.

[ConradIrwin]

@SomeoneToIgnore I think we've decided to wait on any big windows stuff until post linux; so let's not worry about CI.

@kazatsuyu Thanks for working on this. In general we should avoid unwrap'ing on errors, left a suggestion in line, but otherwise good to go.

[ConradIrwin]

Thanks!