Skip to content

upd: Revert "fix #228025 amazon.com - missed trackers (#228079)"#230702

Closed
mu-hun wants to merge 1 commit intomasterfrom
fix-#227687-amazon.com
Closed

upd: Revert "fix #228025 amazon.com - missed trackers (#228079)"#230702
mu-hun wants to merge 1 commit intomasterfrom
fix-#227687-amazon.com

Conversation

@mu-hun
Copy link
Copy Markdown
Member

@mu-hun mu-hun commented May 6, 2026

Prerequisites

To avoid invalid pull requests, please check and confirm following terms

  • This is not an ad/bug report;
  • My code follows the guidelines and syntax of this project;
  • I have performed a self-review of my own changes;
  • My changes do not break web sites, apps and files structure.

What problem does the pull request fix?

If the problem does not fall under any category that is listed here, please write a comment below in corresponding section

  • Missed ads or ad leftovers;
  • Website or app doesn't work properly;
  • AdGuard gets detected on a website;
  • Missed analytics or tracker;
  • Social media buttons — share, like, tweet, etc;
  • Annoyances — pop-ups, cookie warnings, etc;
  • Filters maintenance.

What issue is being fixed?

Enter the issue address

upd #227687 (comment)

Add your comment and screenshots

After these changes, loading of the main content on amazon.com (#229793) and product page content is having issues.

I think these tracker filters are very risky. This reverts commit 5004879.

Terms

  • By submitting this issue, I agree that pull request does not contain private info and all conditions are met

After these changes, loading of the main content on amazon.com (#229793) and product page content (#227687 (comment)) is having issues

I think these tracker filters are very risky

This reverts commit 5004879.
@mu-hun mu-hun requested a review from a team May 6, 2026 04:16
@mu-hun mu-hun self-assigned this May 6, 2026
Copy link
Copy Markdown
Member

@piquark6046 piquark6046 left a comment

Choose a reason for hiding this comment

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

Does applying $redirect modifier them help?

@mu-hun
Copy link
Copy Markdown
Member Author

mu-hun commented May 6, 2026

Okay, I will check it with $redirect=noopjs or $redirect=nooptext

@mu-hun
Copy link
Copy Markdown
Member Author

mu-hun commented May 6, 2026

@piquark6046 Ah, but this comment (#227687 (comment)) can‘t be reproduced on my end.

So, I can't test with the $redirect modifier. Therefore, I think that reverting it is the best solution for now.

How do I do in this case?

@piquark6046
Copy link
Copy Markdown
Member

#230702 (comment)
The report said that the reporter has installed uBO Lite. Can you test it with uBO Lite again?

@mu-hun
Copy link
Copy Markdown
Member Author

mu-hun commented May 6, 2026

Can't be reproduced with uBO Light. Screenshot 2026-05-06 at 4 44 43 PM

@AdguardTeam/filters-maintainers, can anyone reproduce it with uBO Light?

@piquark6046
Copy link
Copy Markdown
Member

Maybe It seems that the issue is occurred by the reporter's specific environmental problem.
I think that we should handle it if there are more data.
By the way, I think that we should await another's opinion.

@mu-hun
Copy link
Copy Markdown
Member Author

mu-hun commented May 6, 2026

Okay, I'm converting this PR to Draft while awaiting.

@mu-hun mu-hun marked this pull request as draft May 6, 2026 10:09
@Alex-302
Copy link
Copy Markdown
Member

Alex-302 commented May 7, 2026

@mu-hun Hm, I can't reproduce also. Maybe we should not remove these rules.

@mu-hun
Copy link
Copy Markdown
Member Author

mu-hun commented May 8, 2026

Okay, I'm closing this PR and close #227687 for this issue.

@mu-hun mu-hun closed this May 8, 2026
@mu-hun mu-hun deleted the fix-#227687-amazon.com branch May 8, 2026 00:47
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.

3 participants