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

Fix rendering, comments and anti-adblock on https://www.gazeta.ru/ #205

Merged
merged 2 commits into from Sep 19, 2019
Merged

Conversation

@ryanbr
Copy link
Collaborator

ryanbr commented Sep 19, 2019

1. Fix site rendering and comments api used by the site:

Visiting https://www.gazeta.ru will cause the site not to render correct. Specifically caused by the block on myqualification.rambler.ru.

Following scripts are being blocked:
https://cdn-comments.rambler.ru/widget.js
https://subscriptions.rambler.ru/subscribe_form/gazeta:header/
https://c.rambler.ru/api/app/3/comments-count?xid=12660673&xid=12660337&xid=12657883&xid=12641047
https://myqualification.rambler.ru/dvwegmyv/YWxpZTcuaHVoem12QHsiZGF0YSI6ey..
https://myqualification.rambler.ru/lnmmhspf/YnNiejAuZHcxNWNqQHsiZGF0YSI6eyJBY3Rpb24iOiJTdGF0QmxvY2tlcnMiLCJzdGF0Ijp7ImVsIjoxLCJyZSI6MSwic2l0ZSI6?juirzbcl=Ind3dy5nYXpldGEucnUifSwidXNlciI6e&mcehw=yJydWlkIjpudWxsLCJscnVpZCI6InBROE&qbzg=FBTE5GZzEzSUU3akFBY3QzY3dBPSJ9fX0&yayv=%3D

The following domains myqualification.rambler.ru, subscriptions.rambler.ru and cdn-comments.rambler.ru and c.rambler.ru/api/app/3/comments-count seems to be legit content. (Comments by rambler.ru). This an overreach by the disconnect blocking rambler.ru.

We're still blocking the rambler.ru trackers in EP: (which are blocked on this site)
||sync.rambler.ru^
||ssp.rambler.ru^

2. Fix Anti-adblock tracker:

Anti adblock Tracking also see:
https://static.gazeta.ru/nm2015/js/advertising.js

Source:
(function (){
window.ADBLOCK_DETECTED = false;
})();

@pes10k
pes10k approved these changes Sep 19, 2019
@pes10k pes10k merged commit 4254214 into master Sep 19, 2019
@pes10k pes10k deleted the gazeta branch Sep 19, 2019
@ryanbr ryanbr mentioned this pull request Sep 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.