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

presumably wrong domains #154

Closed
ghost opened this issue Nov 14, 2020 · 12 comments
Closed

presumably wrong domains #154

ghost opened this issue Nov 14, 2020 · 12 comments
Assignees
Labels
bug Something isn't working waiting for Mitch

Comments

@ghost
Copy link

ghost commented Nov 14, 2020

@rusty-snake commented on Nov 14, 2020, 5:40 PM UTC:

ATM there are three domains which did not look right.

  • ://www.bit.ly/38tqwJT
  • blog.goo.ne.jp/orochima/
  • freeiv.pixnet.net/blog

This issue was moved by dnmTX from Ultimate-Hosts-Blacklist/Ultimate.Hosts.Blacklist#593.

@dnmTX dnmTX added the bug Something isn't working label Nov 14, 2020
@Somebodyisnobody
Copy link
Member

Somebodyisnobody commented Nov 14, 2020

Seems like we have structural problems with input validation. What's about implementing a regex filter?
^([\w0-9-]+\.)*[\w]{2,10}$
.. 2nd-lvl-domain |.......... tld

@Somebodyisnobody
Copy link
Member

@dnmTX it's also an upstream issue (when it's not affected by the caching-bug). Can you tag it?

@dnmTX
Copy link
Contributor

dnmTX commented Nov 14, 2020

Thanks for the report @rusty-snake.
First there is a bug/problem with the filtering/update and even if there are any changes in upstream they will not make it here until it's fixed.
Second:
blog.goo.ne.jp/orochima/ and freeiv.pixnet.net/blog are present in Bad_JAV_Sites and the origin of that list is clearly unknown(info.json is empty). @funilrys will need more info from you on this one and advise on what's next. Possible removal maybe?

://www.bit.ly/38tqwJT is present in xorcan_tuerk_ad_list
and checking upstream it's not as bad but it's clearly needs to be looked at.
PING @xorcan for further action/fix in upstream 👍
@xorcan just FYI you CAN'T USE ! for comments in a hosts file,only # is permited. Please fix this as well!

Guys,thanks for the reports but until @funilrys fix the filtering/update issue nothing else can be done at this moment.
Be patient, once it's fixed will start digging into all those issues to make everyone's experience better. Thank you!

dnmTX added a commit to Ultimate-Hosts-Blacklist/Bad_JAV_Sites that referenced this issue Nov 14, 2020
dnmTX added a commit to Ultimate-Hosts-Blacklist/Bad_JAV_Sites that referenced this issue Nov 14, 2020
dnmTX added a commit to Ultimate-Hosts-Blacklist/Bad_JAV_Sites that referenced this issue Nov 14, 2020
dnmTX added a commit to Ultimate-Hosts-Blacklist/Bad_JAV_Sites that referenced this issue Nov 14, 2020
@dnmTX
Copy link
Contributor

dnmTX commented Nov 14, 2020

Ok. Considering the fact that the filtering and upstream updating is completely broken at this moment i went ahead and removed all the mentioned domains from their respective lists/repos. It's temporary patch for now,let's hope it will hold.
Will keep this issue open for the time being.

@rusty-snake
Copy link

I discovered this while developing uhb2dnsmasq, where I use ^[0-9A-Za-z._-]{1,500}$ to ensure that no option can be injected into the dnsmasq configuration (e.g. some/\nlisten-address 192.168.0.23) and print mismatches to see if this regex is to strict.

Seems like we have structural problems with input validation. What's about implementing a regex filter?

Is there any input validation? Or would it be possible to inject something like doubleclick.net\n1.2.3.4 google.com?

@dnmTX
Copy link
Contributor

dnmTX commented Nov 14, 2020

Seems like we have structural problems with input validation. What's about implementing a regex filter?

@rusty-snake @Somebodyisnobody such a issues should be addressed in dev-center but let's take one problem at a time and put that filtering problem as a high priority due to none of us here had any of their lists updated properly for almost a year. Every other issue could/should wait.

@ghost
Copy link

ghost commented Jan 17, 2021

i fixed all wrong things. thank all of you for report me. take a look again please: https://github.com/xorcan/hosts/blob/master/README-EN.md

@dnmTX @Somebodyisnobody

@dnmTX
Copy link
Contributor

dnmTX commented Mar 13, 2021

@funilrys Bad_JAV_Sites has no upstream link.
Are we keeping it and maintain it ourselfs or needs to go?

@funilrys
Copy link
Member

@dnmTX let's ping @mitchellkrogza, those without upstream link exist because of him.

@dnmTX
Copy link
Contributor

dnmTX commented Mar 13, 2021

those without upstream link

How many we talking about here?

@funilrys
Copy link
Member

@dnmTX

Will tell you once back at my computer but a simple GitHub search of "raw_link": null or "raw_link": "" in our whole organisation should tell us that...


Sent from my supposedly smart Phone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working waiting for Mitch
Projects
None yet
Development

No branches or pull requests

5 participants