-
Notifications
You must be signed in to change notification settings - Fork 71
parsing fonts.googleapis.com incorrect #4
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
Comments
Mhmm ... that's tricky, because That's why Browser vendors, however, use publicsuffix as source to determine whether the entered string is a URL or a search keyword and even for restricting cookie access. Honestly, I don't know how to proceed. Can we define an expected behavior? Maybe publicsuffix is the wrong source, maybe we should leverage DNS? |
That's interesting, I didn't know the concept of public suffix. I used this module to separate the domain name and use whois to check who own this domain. I think that maybe it would be best to add the domain LTD and the domain publicsuffix. for example:
|
Seems I am having the same issue as above #6. It would be great to seperate out publicsuffix, because honestly that's just a guess and we will never catch them all. Where as tld is very clear, there can only be a finite number of tld's and in my case I'm most interested in the tld, domain, subdomain breakdown. |
Well, as far as I can tell, is the TLD just the last portion of the URL. I don't know if there is a rule that separates the However, since browser vendors use this list even for restricting cookie access, it's not just an arbitrary list. But I have to admit that it's somewhat surprising... I'm thinking about using DNS resolution to distinct between |
parse-domain excludes private domains by default now. Shipped with |
parsing
fonts.googleapis.com
isn't work correctly. the subdomain should befonts
the domain should begoogleapis
and the tld should becom
.this is what I got while parsing the domain
{ tld: 'googleapis.com', domain: 'fonts', subdomain: '' }
The text was updated successfully, but these errors were encountered: