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

include complete redirect rule feature #59

Closed
wants to merge 6 commits into from
Closed

Conversation

x0day
Copy link

@x0day x0day commented Feb 16, 2022

hi, thanks for this project first, it helps me a lot.

in this PR, I complete some feature about scriptlet injection and redirect rules.

  1. update adblock-rust => 0.4.3
  2. add include_redirect_urls parameter for FilterSet.add_filter_list and FilterSet.add_filters
  3. add Engine.add_resource method.
  4. add adblock.AddResourceError Exception
  5. add redirect_type for BlockerResult
  6. add some tests for these functions.

dm added 3 commits February 16, 2022 10:34
1. update adblock-rust => 0.4.3
2. add `include_redirect_urls` parameter for `FilterSet.add_filter_list` and `FilterSet.add_filters`
3. add `Engine.add_resource` method.
4. add `adblock.AddResourceError` Exception
5. add `redirect_type` for `BlockerResult`
6. add some tests for these functions.
@x0day
Copy link
Author

x0day commented Feb 16, 2022

Some implementations refer to the https://github.com/brave/adblock-rust-ffi.

@ArniDagur ArniDagur self-requested a review February 16, 2022 14:46
src/lib.rs Outdated Show resolved Hide resolved
tests/test_exceptions.py Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
@x0day x0day requested a review from ArniDagur February 20, 2022 03:26
@x0day
Copy link
Author

x0day commented Feb 22, 2022

@ArniDagur still have something not ready to merge?

@ArniDagur
Copy link
Owner

Thank you for your contribution. This has been merged as #60.

@ArniDagur ArniDagur closed this Feb 26, 2022
@x0day
Copy link
Author

x0day commented Feb 27, 2022

Thank you for your contribution. This has been merged as #60.

can you release a new version on pypi.org? thank you.

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Jun 16, 2022
https://build.opensuse.org/request/show/982852
by user mia + dimstar_suse
- Update to 0.5.2
  * Include complete redirect rule feature
    (gh#ArniDagur/python-adblock#59)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants