Skip to content
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

「Check URL is valid URL」 is invalid #9

Closed
KageShiron opened this issue Dec 16, 2022 · 2 comments
Closed

「Check URL is valid URL」 is invalid #9

KageShiron opened this issue Dec 16, 2022 · 2 comments
Labels
Status: Proposal Request for comments

Comments

@KageShiron
Copy link

大変興味深い記事をありがとうございます。

Check URL is valid URL throw an error when parsing invalid url string.

こちらの記述は、必ずしも適切な表現ではないかと思います。
明確にInvalidな様々なURLを、ブラウザは例外を出さずにパースします。

new URL("http://;;;");

上記の処理は、ChromeではTypeErrorとなり、Firefoxではhttp://;;;というURLとしてパースが成功します。
本当にValidなURLかどうかを判断する方法はおそらくこの世に存在しないため、「InvalidなURLの一部をエラーにできる」といった表現が穏当でしょうか…。

@azu azu added the Status: Proposal Request for comments label Dec 16, 2022
@azu
Copy link
Owner

azu commented Dec 16, 2022

validが曖昧な感じですかね。
Absolute-URL 絶対URLかどうかの判定が良さそうですかね。(URLとしてパースできる は流石になんか意味ない感じに見えてしまう)
https://url.spec.whatwg.org/#constructors

用途はschemeが無かったりhttp//みたいな書きミスを見つけるぐらいで、
実際に入力されたURLを判定するとかの場合は https://github.com/azu/url-cheatsheet#check-url-is-http の方を使うという感じな気もしますね。
(httpじゃないURLを扱うケースはそこまで多くもないので)

@azu
Copy link
Owner

azu commented Dec 16, 2022

whatwg/url#713 でも似た話がありました

@azu azu closed this as completed in 9ed5585 Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Proposal Request for comments
Projects
None yet
Development

No branches or pull requests

2 participants