iploggerfilter icon indicating copy to clipboard operation
iploggerfilter copied to clipboard

Idna compliant

Open spirillen opened this issue 2 years ago • 4 comments

Tested with PyFunceble

Kept the lists of INACTIVE + INVALID as requested in and https://github.com/piperun/iploggerfilter/issues/49

A new category have also been added at the bottom of the filterlist with the IDNA records sorted by subcategory.

Closes https://github.com/piperun/iploggerfilter/issues/50

Closes https://github.com/piperun/iploggerfilter/issues/49

spirillen avatar Jan 02 '22 23:01 spirillen

Just to have a second look on these records @D4v1dH03 tested them as well, even I only expected a selective test, it turned out he tested them all, Lovely guy.

https://github.com/easylist/easylist/issues/10318#issuecomment-1003798436

spirillen avatar Jan 03 '22 01:01 spirillen

Looks alright, you wouldn't mind adding the test code you use to look into if the DNS is active/invalid?

piperun avatar Jan 04 '22 00:01 piperun

Sure, it is based on our (@funilrys and me) Pyfunceble templates They might need some tweaking to match pyfunceble version 4.x

But in your case the command line code should be looking something like this:

Installing PyFunceble

Dev (Usual rather stable and the only one I use)
python3.10 -m pip install -I -U --pre pyfunceble-dev

Stable
python3.10 -m pip install -I -U pyfunceble

Executing PyFunceble test from command line

Then copy this .PyFunceble.overwrite.yaml to ~/.config/PyFunceble/.PyFunceble.overwrite.yaml Change values as you sees fit

Here is the relevant docs https://pyfunceble.readthedocs.io/en/dev/usage/index.html#from-a-terminal

pyfunceble --adblock -f filterlist Test_results/inactive.txt

That is actually about it

The pitfall in this is your adblock rules are not copied, but as you are using one rule, this is fastly added by a script.

The un-official docs repo is here https://mypdns.org/spirillen/PyFunceble

spirillen avatar Jan 04 '22 01:01 spirillen

After a very long time (sorry about that), I think this should work.

@rebase needed

spirillen avatar Aug 02 '22 11:08 spirillen