Lower-case TLD lookup - Githubissues

jpillora / go-tld

TLD Parser in Go

MIT License

125 stars 18 forks source link

Lower-case TLD lookup #9

Closed jmdacruz closed 3 years ago

jmdacruz commented 3 years ago

It looks like PublicSuffix is case sensitive when looking up TLDs, so google.com is ICANN-managed but google.Com is not, which is not correct (both are equivalent). This PR addresses this issue by lower-casing the domain before calling PublicSuffix

jmdacruz commented 3 years ago

It looks like my PR was not probably rebased on master... I'll create a new PR from a clean branch instead. See #10