Re: [whatwg/fetch] Block 'ftp:' requests from non-'ftp:' clients. (#464)

Hey @annevk. This seemed like a reasonable spot to put the FTP check we briefly discussed on blink-dev@. I don't think the ordering particularly matters, except insofar as it should come after CSP so that violation reports are generated correctly. I considered putting the check into the `ftp` case of Basic Fetch's switch, but that seemed a little more hidden than reasonable. WDYT?

As noted on IRC, tests will be a little annoying, as WPT doesn't actually support FTP, and @jgraham (reasonably) doesn't consider such support to be a priority. I'll put something together, but I think it'll be ugly.

-- 
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/fetch/pull/464#issuecomment-274750319