Adblock Plus

Posted: **Sat Feb 11, 2012 8:48 pm**

Hi,

Not a very big bug as I doubt anyone will use trailing-dot-domain URLs, but worth mentioning:
If you visit http://torrentz.eu./ (or any random http URL and just append a dot to the domain) the following bugs can be found:
1. the EHH suggests the domain ""
2. the generated domain-specific filter from the EHH (foo.bar.##anything) won't apply to the page, (neither will foo.bar##anything).

Kind regards,
Famlam

Posted: **Mon Feb 13, 2012 10:37 pm**

The correct rule would be indeed "foo.bar.##anything" - Gecko treats "foo.bar." and "foo.bar" as different domains.

Now there are apparently two issues in how we are handling this domain name, will look into it.

Posted: **Sat Jun 23, 2012 3:50 pm**

Fixed:
https://hg.adblockplus.org/elemhidehelp ... 9e862d1a07
https://hg.adblockplus.org/adblockplus/rev/a2b0c27c133d

Posted: **Sat Jun 23, 2012 4:07 pm**

Thanks!

FYI, from your patch (line 1.87):

Code: Select all

if (this.ignoreTrailingDot)
  docDomain = docDomain.replace(/\.+$/, "").toUpperCase();
docDomain = docDomain.toUpperCase();

Shouldn't that be

Code: Select all

if (this.ignoreTrailingDot)
  docDomain = docDomain.replace(/\.+$/, "");
docDomain = docDomain.toUpperCase();

as you are uppercasing twice?

Posted: **Sat Jun 23, 2012 4:59 pm**

You are correct, thanks for noticing!

Fixed: https://hg.adblockplus.org/adblockplus/rev/db19b081bd9c

Adblock Plus

[Fixed] Domain with a trailing dot: hiding rules won't apply

[Fixed] Domain with a trailing dot: hiding rules won't apply

Re: Domains with a trailing dot: hiding rules won't apply

Re: Domains with a trailing dot: hiding rules won't apply

Re: [Fixed] Domain with a trailing dot: hiding rules won't apply

Re: [Fixed] Domain with a trailing dot: hiding rules won't apply