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

AdNauseam updating filters slower than uBlock? #2420

Closed
mneunomne opened this issue Oct 9, 2023 · 8 comments
Closed

AdNauseam updating filters slower than uBlock? #2420

mneunomne opened this issue Oct 9, 2023 · 8 comments

Comments

@mneunomne
Copy link
Collaborator

This is just an impression I have at the moment, based on the user's feedback. Need to make proper tests.

One possibility is the fact uBlock users the assets/assets.json on their master remote, and we use it on ours, which is always outdates to the last merge. Not sure what is the correct approach since we had problems in the past when we got stuck too far behind, and the assets on uBlock master started to be too far ahead of our code.

@dhowe
Copy link
Owner

dhowe commented Oct 13, 2023

labelled as high-priority in relation to youtube-video issues

@dhowe
Copy link
Owner

dhowe commented Oct 18, 2023

@mneunomne see also #2425

@mneunomne
Copy link
Collaborator Author

@dhowe maybe we should add a warning on youtube for the users to keep their list updated to prevent ads?

@mneunomne
Copy link
Collaborator Author

Regarding the assets.json, I'm thinkling if there is a way for us to always use the uBlock assets.json instead of ours, and just inject adnauseam asset object in the list. Currently because we have to reference our own, we tend to get outdated if the assets get changed.

{
"adnauseam-filters": {
		"content": "filters",
		"group": "default",
		"title": "AdNauseam filters",
		"tags": "ads",
		"contentURL": [
			"https://raw.githubusercontent.com/dhowe/AdNauseam/master/filters/adnauseam.txt",
			"assets/ublock/adnauseam.txt"
		],
		"cdnURLs": [
			"https://cdn.jsdelivr.net/gh/dhowe/AdNauseam@master/filters/adnauseam.txt",
			"https://cdn.statically.io/gh/dhowe/AdNauseam/master/filters/adnauseam.txt"
		],
		"supportURL": "https://github.com/dhowe/adnauseam/issues"
	},
}

@dhowe
Copy link
Owner

dhowe commented Oct 18, 2023

when would this update/inject happen ?

mneunomne added a commit that referenced this issue Oct 18, 2023
set assets.json ref as adnauseam master
@mneunomne
Copy link
Collaborator Author

What I am implenting is that the reference of the assets.json to be from adnauseam@master. Which means that if we want to keep up-to-date with uBlock assets filter list, then we need to make sure is that the merge from uBlock Origin are made more frequently, also with their beta-releases, so that our master branch is as close-as-possible from theirs.

If we inject the adnauseam filter to the ublock@master then we wouldnt have this need, but we would need to understand well how the asset management system how it currently works. For now the solution of having it referencing to adnauseam@master makes more sense. We can have the injection solution as a possible future improvement.

@mneunomne
Copy link
Collaborator Author

I have tested now, and it seems that AdNauseam filter are updating quite fast. I am not sure which of our changes have improved this - either (#2420 or #2473).

@mneunomne
Copy link
Collaborator Author

Fixed, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants