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

Add List-KR DNS #135

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add List-KR DNS #135

wants to merge 4 commits into from

Conversation

usb9245
Copy link

@usb9245 usb9245 commented Feb 15, 2024

#31

@usb9245 usb9245 changed the title Add List-KR DNS Add List-KR Feb 15, 2024
@piquark6046
Copy link

For blocking DNS purpose, please delete all other URLs except https://cdn.jsdelivr.net/gh/adguardteam/HostlistsRegistry@main/assets/filter_25.txt.

@usb9245 usb9245 changed the title Add List-KR Add List-KR DNS Feb 15, 2024
@1201ysy
Copy link

1201ysy commented Apr 19, 2024

Please add

@jacklee-5
Copy link

please approve this PR
is there any more issues?

For blocking DNS purpose, please delete all other URLs except https://cdn.jsdelivr.net/gh/adguardteam/HostlistsRegistry@main/assets/filter_25.txt.

@piquark6046
Copy link

The all other URLs except the hostlist can causes an incorrect blocking because of incorrect parsing syntax.

@jacklee-5
Copy link

@piquark6046 I think the json file has only one URL now. Could you review it again?

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.

None yet

4 participants