Valid TLD Not Parsed #82
Comments
|
You will have to convert from punycode back to utf since the tld list is in utf. @jhnns correct? |
|
Thanks for reporting this. Yes, with the current version, you will need to translate the Punycode to Unicode because the internal list of domain names is using Unicode. However, this will change with the next major version bump (see #14). We will only accept sanitized hostnames (basically those from |
jhnns
added a commit
that referenced
this issue
Apr 23, 2020
…ements - Run against [psl example domains](https://raw.githubusercontent.com/publicsuffix/list/master/tests/test_psl.txt). Closes #1 - Add support for international domain names. Fixes #16 #82 and #44 - Only accept hostnames instead of whole URLs. Fixes #49 and #14 - Do not auto update tries on npm install. Fixes #42 #48 and #90 - Use "node-fetch" instead of "got". Fixes #78 and #62 - Use Node's "assert" module instead of Jest for smoke test. Fixes #92 #93 #89 #91 - Recognize IPv4 and IPv6 in hostnames. Fixes #102 BREAKING CHANGE: This release is a complete rewrite in TypeScript. It fixes some long outstanding bugs and comes with improvements we were planning for quite some time. The major changes are: 1. parseDomain does not accept whole URLs anymore. Only the hostname section of a URL is allowed now. 2. We removed the options object. Custom TLDs are returned as "valid but not listed". The parse result contains both the result with private TLDs and without private TLDs. 3. Dropped Node 6 support. We recommend reading the README since the public API as changed quite a lot.
|
Fixed with |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
parseDomain(""xn--90abvrccwbp.xn--p1acf")The above will return
null..xn--p1acfis a valid TLD -> https://www.iana.org/domains/root/db/xn--p1acf.htmlThe text was updated successfully, but these errors were encountered: