tld check should be disabled by default

[sindresorhus]

Or preferably just removed.

New tld's are coming out all the time and it's not feasible to keep it update to date. Even if you can, people using it won't and it will lead to annoying websites that doesn't accept yo@cat.ninja...

[kevva]

This is so we can support URLs without protocols 25050cc. Without it would make this module quite loose, accepting pretty much anything with a . in it. We could make protocols mandatory again though.

[sindresorhus]

Alright, I think we need at least an option then. Depending on the use case. If I want to use this regex to find URLs, I want to use the tld list, but only for matching URLs without protocol. Meaning, two patterns in one, one with tld list for only matching schemeless URLs and one without tld list for matching protocol URLs. I also have the use-case of wanting to "validate" and URL, which is kinda the use-case for the exact option. Here I definitely don't want the tld list.

[kevva]

Yup, that sounds good to me. So the exact option would need a protocol then, right? I guess we won't need an option if we're just matching two pattterns. One with a protocol and without TLDs and one without a protocol and with TLDs. The exact option would just use the former.

[sindresorhus]

So the exact option would need a protocol then, right?

I was asking myself that and not sure. That's why I didn't mention it. I guess so.

Yes to everything else.

[kevva]

It's a little weird, because exact is just meant to match the exact string. I wonder if we don't have to introduce a strict option regardless.

[sindresorhus]

Hmm, I need to think about that. Let's revisit in a few days.

[kevva]

ping @sindresorhus