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

I want to contribute coin-miner filter list to uBlock please #785

Closed
PeterDaveHello opened this issue Oct 26, 2017 · 2 comments
Closed

I want to contribute coin-miner filter list to uBlock please #785

PeterDaveHello opened this issue Oct 26, 2017 · 2 comments

Comments

@PeterDaveHello
Copy link

PeterDaveHello commented Oct 26, 2017

Hi there,

I've noticed that pull request is not welcome as the main project CONTRIBUTING mentions, just want to make sure if it possible or how to contribute to the filter list, as I see this uAssets repo but I didn't see the filter list in this repo.

I've make some changes in my own forked uBlock repo:

Commits:

Branch:

Which would add "NoCoin adblock list"(https://github.com/hoshsadiq/adblock-nocoin-list) in the filter list, would you please let me know how can I do it or I just can't?

Many thanks!

@okiehsch
Copy link
Contributor

okiehsch commented Oct 26, 2017

If you think uBlock should add a filter list, you can open an issue at https://github.com/gorhill/uBlock.
It has been done before. gorhill/uBlock#3089.

I see no reason to add https://github.com/hoshsadiq/adblock-nocoin-list, because at a quick glance all entries except ||api.inwemo.com^$third-party are already included in https://github.com/uBlockOrigin/uAssets/blob/master/filters/resource-abuse.txt, which is enabled by default in uBO.
inwemo.com has a mandatory opt-in as far as I can see, so unless someone can provide an example of it starting mining without user interaction I don't think it should be added to resource-abuse.txt, as I already stated in another case.

@PeterDaveHello
Copy link
Author

@okiehsch thanks for your prompt reply, I'll take a look at the resource you gave me 😄

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

No branches or pull requests

2 participants