Re: [whatwg/url] Allow ports in file URLs (#548)

Allright, so since I commented, I decided I should have a look. I have removed the file host state in favour of the authority state in the reference implementation. 

Changes:
1. The host parser can now return an empty host. 
2. The file host state has been removed. 
3. The file authority is parsed using the generic authority parser. 

Currently, there are still checks within the authority parser that return failure if a file URL has credentials or a port. 
This can be easily changed to match whatever is decided for this issue. 

Actually, 1. may be done in a different way too. And as I write this, I wonder if file URLs should be allowed to have hosts that are converted to an empty string by domainToASCII. 


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/url/issues/548#issuecomment-707874798

Received on Tuesday, 13 October 2020 16:51:36 UTC