rushmorem / publicsuffix

An implementation of Mozilla's Public Suffix List in Rust
MIT License
96 stars 17 forks source link

Don't assert known suffix for wildcard match #31

Closed bradfier closed 3 years ago

bradfier commented 4 years ago

Where the domain was only successfully matched by the wildcard match rule, we shouldn't assert that the domain has a known suffix, nor that it is definitely an ICANN or Private domain.

This could also arguably be applied to the 'registrable' portion of the Domain, given a wildcard only match tells us nothing about the registrability of the last token.

However since the registrable field is conflated with the root of the domain, we leave it alone so fewer changes are required elsewhere in the library.