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

tvr.by - the site is broken by AdGuard injections #1076

Closed
Alex-302 opened this issue Aug 6, 2019 · 13 comments
Closed

tvr.by - the site is broken by AdGuard injections #1076

Alex-302 opened this issue Aug 6, 2019 · 13 comments
Assignees
Milestone

Comments

@Alex-302
Copy link
Member

Alex-302 commented Aug 6, 2019

With AdGuard codepage is wrong.

image

Not reproduced in Firefox, Opera 12 and IE.

This exclusion helps:
@@||tvr.by^$jsinject,elemhide
this - no:
@@||tvr.by^$jsinject,generichide

AdGuard 7.1.2874
Chrome and Opera 62

@ameshkov ameshkov added this to the v1.5 milestone Aug 6, 2019
@zubrRB
Copy link

zubrRB commented Aug 6, 2019

В Firefox при вводе адреса в адресную панель во вкладке на мгновение видны кракозябры, но затем сайт открывается нормально.

@zubrRB
Copy link

zubrRB commented Aug 10, 2019

Ещё несколько недель назад сайт открывался нормально. AdGuard с тех пор не обновлялся, поэтому всё дело, вероятно, в последних обновлениях хромообразных браузеров.

@vshlobinag
Copy link

Can't reproduce on nightly 7.2.2931 (cl 1.4.163)

@Alex-302
Copy link
Member Author

@zubrRB проверь снова. У меня тоже не повторяется.

@zubrRB
Copy link

zubrRB commented Sep 19, 2019

На nightly 7.2.2931 (cl 1.4.163) и Chrome 77.0.3865.90 - повторяется. Ломает связка "Скрыть User-Agent" (по умолчанию) + использование любого из двух юзерскриптов:
SaveFrom.net helper https://download.sf-helper.com/chrome/helper.user.js
Mouseover Popup Image Viewer https://greasyfork.org/ru/scripts/404-mouseover-popup-image-viewer
Причем отключение только раздела Антитрекинга или только обоих юзерскриптов проблему не решает, требуется одновременное отключение сокрытия юзерагента и одного из перечисленных юзерскриптов (любого из них; какого именно значения не имеет).

@zubrRB
Copy link

zubrRB commented Sep 19, 2019

Либо перевод ползунка Антитрекинга в положение - Отключен, тогда даже юзерскрипты отключать нет необходимости. При этом ручное отключение всех разделов Антитрекинга не помогает, только полное отключение его как функции AdGuard.

@vshlobinag
Copy link

Воспроизводится с одним скриптом SaveFrom.net helper, но нет необходимости отдельно отключать "скрыть UA" с отключенным Антитрекингом чтобы переставало воспроизводится (возможно потребуется несколько раз обновить страницу с отключенным кешом).

@zubrRB
Copy link

zubrRB commented Sep 19, 2019

Я об это и указал: отключение сокрытия UA+скрипта либо лишь только отключение Антитрекинга.

@vshlobinag
Copy link

Есть три способа отключить Антитрекинг (при условии, что изначально включена только одна его опция "Скрыть UA")

  1. Отключить опцию "Скрыть UA" и оставить включенным сам Антитрекинг.
  2. Отключить опцию "Скрыть UA" и отключить Антитрекинг.
  3. Оставить включенным "Скрыть UA" и отключить Антитрекинг.

Я не могу воспроизвести так, чтобы проявлялся баг в каком-либо из этих случаях. Но в сообщении от Sep 19, 2019 утверждается "При этом ручное отключение всех разделов Антитрекинга не помогает, только полное отключение его как функции AdGuard.", что соответствует утверждению, что способы 1 и 3 дают баг и только 1 не дает ему воспроизвестись. Возможно я не так понял это предложение.

У меня такое не воспроизводится на Chrome c отлюченным кешом в меню разработчика во вкладке Network. Чтобы перестало воспроизводится хватает двух обновлений страницы, либо подождать секунд 30.

Основной баг воспроизводится.

@zubrRB
Copy link

zubrRB commented Sep 20, 2019

Включаете Антитрекинг, опцию в нём "Скрыть UA" и устанавливаете оба скрипта. После чего либо:

  • Отключить опцию "Скрыть UA" и один любой из скриптов.
  • Отключить Антитрекинг.

При изменении любой из настроек следует дождаться перезапуска защиты (изменения цвета значка в трее с зеленого на желтый и снова на зелёный). Тестировалось в т. ч. в режиме инкогнито.

@vshlobinag
Copy link

Я просто к тому, что для воспроизведения бага достаточно двух условий: один скрипт и включить "Скрыть AU" любым способом :) Получается проще формулировка.

Например в способе "Отключить опцию "Скрыть UA" и один любой из скриптов." я просто отключил "Скрыть UA" и у меня перестал баг проявляться. У меня по крайней мере так.

Но это детали. Суть понятна, буду смотреть. Спасибо.

@zubrRB
Copy link

zubrRB commented Sep 20, 2019

Для чистоты эксперимента вот все мои настройки AdguardTeam/AdguardFilters#40623

@vshlobinag
Copy link

vshlobinag commented Sep 24, 2019

Declined: core/pull-requests/1521
New: core/pull-requests/1527

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

6 participants