Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Network filter domain option subsetting #7

Merged
merged 1 commit into from May 9, 2019
Merged

Conversation

@AndriusA
Copy link
Collaborator

AndriusA commented May 8, 2019

Updates network filter tests to include all permutations of domain subsetting cases, mirroring uBlock Origin behaviour.

I.e. in the case of filter adv$domain=~example.com|foo.example.com and request URL = http://example.net/adv it does not match when issued from:

Closes #5

…bsetting cases, mirroring uBlock Origin behaviour.

I.e. in the case of filter `adv$domain=~example.com|foo.example.com` and request URL = `http://example.net/adv` it does not match when issued from:

- http://example.com
- http://foo.example.com
- http://subfoo.foo.example.com
- http://bar.example.com
- http://anotherexample.com

Closes #5
@bbondy
bbondy approved these changes May 9, 2019
@AndriusA AndriusA merged commit 1915c47 into master May 9, 2019
@AndriusA AndriusA deleted the domain-subsetting branch May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.