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

A: exception for doubleclick redirects from finder.com.au and finder.com #18276

Merged
merged 2 commits into from Jan 20, 2024
Merged

Conversation

fmrk
Copy link
Contributor

@fmrk fmrk commented Jan 12, 2024

Adding Exception for Finder Domains
This pull request proposes an update to the existing doubleclick.net filter exception rule to include Finder domains finder.com.au, finder.com and creditcard.com.au.
This change addresses the issue where ad blockers were inadvertently closing user-intentionally opened tabs due to redirects involving ad.doubleclick.net.
The issue has been first detected while using uBlock Origin and discussed also here: uBlockOrigin/uAssets#21939

Copy link

@mopielka mopielka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I support adding finder.com.au to the list of exceptions. On our site, the redirection process to a partner's site is a deliberate action by the user. It requires the user to click a 'Go to site' button, after which they are presented with a redirect page before being taken to the partner’s site. This process is intentional and cannot occur by accident. Therefore, maintaining this behavior is crucial, as it aligns with the intended user experience on finder.com.au. Implementing a mechanism that interrupts this process would be counterproductive to the site's functionality and user engagement.

@monzta monzta merged commit 2f533a2 into easylist:master Jan 20, 2024
@fmrk fmrk deleted the finder branch January 22, 2024 16:22
@fmrk fmrk mentioned this pull request Feb 9, 2024
@fmrk fmrk mentioned this pull request Mar 27, 2024
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

3 participants