Skip to content
This repository was archived by the owner on Nov 15, 2017. It is now read-only.

Commit 3668036

Browse files
committed
this fixes #239
1 parent 1c913a4 commit 3668036

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

js/abp-filters.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,10 +36,10 @@ var reIgnoreElementHide = /##|@#/;
3636
var reIgnoreWhitelist = /^@@/;
3737
var reIgnoreFilter = /^\|http/;
3838
var reConditionalRule = /\$/;
39-
var reHostnameRule = /^\|\|[a-z0-9.-]+[a-z0-9]\^?$/;
39+
var reHostnameRule = /^\|\|[0-9a-z.-]+[0-9a-z]\^?$/;
4040
var reHostnameToken = /^[0-9a-z]+/g;
41-
var reGoodToken = /[%0-9A-Za-z]{2,}/g;
42-
var reAnyToken = /[%0-9A-Za-z]+/g;
41+
var reGoodToken = /[%0-9a-z]{2,}/g;
42+
var reAnyToken = /[%0-9a-z]+/g;
4343
var reThirdPartyCondition = /\$third-party$/;
4444

4545
// My favorite regex tester: http://www.gethifi.com/tools/regex#
@@ -600,6 +600,12 @@ var matchString = function(url, srcDomain, dstHostname) {
600600
//adbProfiler.countUrl();
601601
//adbProfiler.testCounter(true);
602602

603+
// https://github.com/gorhill/httpswitchboard/issues/239
604+
// Convert url to lower case:
605+
// `match-case` option not supported, but then, I saw only one
606+
// occurrence of it in all the supported lists (bulgaria list).
607+
url = url.toLowerCase();
608+
603609
var matches, f;
604610
var tokenBeg, tokenEnd;
605611
var thirdParty = dstHostname.lastIndexOf(srcDomain) !== (dstHostname.length - srcDomain.length);

0 commit comments

Comments
 (0)