Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: lib/abp2blocklist.js

Issue 29349790: Issue 4326 - || should match subdomains too (Closed)
Patch Set: Created Aug. 12, 2016, 2:31 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | test/abp2blocklist.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/abp2blocklist.js
diff --git a/lib/abp2blocklist.js b/lib/abp2blocklist.js
index aa074f5acfaac19b5552a72d8b942b66198441a9..0309dd21bc22fce2d68f896028bfba32c5d2715a 100644
--- a/lib/abp2blocklist.js
+++ b/lib/abp2blocklist.js
@@ -153,7 +153,7 @@ function parseFilterRegexpSource(text)
{
hostnameStart = i + 1;
canSafelyMatchAsLowercase = true;
- regexp.push("https?://");
+ regexp.push("https?://([^/]+\\.)?");
break;
}
regexp.push("\\|");
« no previous file with comments | « no previous file | test/abp2blocklist.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld