bug(Parser): incorrect result of getRegistrableDomain() about tldextract

[layershifter]

Hi @atypicalprogrammer

TLDExtract uses TLDDatabase which uses PublicSuffix List as single source of truth. PSL has two sections: ICANN and private domains. So, when you set TLDExtract's parsing mode it operates with determined sections.

blogspot.com is defined in private section, so it's private domain. If don't need private domains in your result, you need to set parsing mode:

$extract = new Extract(null, null, Extract::MODE_ALLOW_ICCAN);
$result = $extract->parse('test.github.com');
// of in fuction
tld_extract('test.github.com', Extract::MODE_ALLOW_ICCAN);

---

Howerer, I agree that getRegistrableDomain() returns an incorrect result without parsing options, it's a priority issue for next major version.

[dgram]

hi,

there are also some other domains falsley valid.
ex.: #test.com or test.com#test_test

->isValidDomain() gives me true for both, and also in the ->getRegistrableDomain() i get the false domain

regards

[layershifter]

@dgram Thanks, I opened #15 for this.