AdguardTeam / AdguardForWindows

AdGuard for Windows open bug tracker
https://adguard.com/
690 stars 72 forks source link

Extensions update themselves with check interval disabled #4004

Closed ghost closed 1 year ago

ghost commented 2 years ago

Так понимаю, что обновление расширений живет своей жизнью?

На всякий случай: Мы получили ваше обращение. Его номер – 554644.

adbuker commented 2 years ago

@GdeZsqyAY, настройка, на которую вы указали, относится к обновлению фильтров, как и следует из названия. Кастомизация периода проверки обновления пользовательских скриптов не предусмотрена. Проверка обновления скриптов проходит раз в 24 часа

zubrRB commented 2 years ago

@northis в марте писал, что проверка обновления скриптов - спустя 30 минут после запуска и потом каждые 30 минут.

ghost commented 2 years ago

@zubrRB

@northis в марте писал, что проверка обновления скриптов - спустя 30 минут после запуска и потом каждые 30 минут.

С того времени у них вышло три версии AdGuard - 7.7 + 7.7.1 +7.7.2

Может в них они чего и подкрутили.

northis commented 2 years ago

@zubrRB 30 минут с момента запуска, далее раз в 24 часа

ghost commented 2 years ago

@northis

Раз скрипты обновляются автоматически, то может уберете их из окошка года делаешь ручное обновление. Смысла их отображения в нем их нет.

zubrRB commented 2 years ago

С тех пор что-то изменилось или так всегда было? https://github.com/AdguardTeam/AdguardForWindows/issues/3718#issuecomment-806707291 Я просто писал, что скрипты у меня почему-то обновляются при ручной проверке. И если автопроверка происходит раз в сутки, то так становится понятнее. А эта проверка привязана к запуску ОС/AG, если ПК работает меньше 24 часов?

zubrRB commented 2 years ago

Я лично всегда ночью задаю ручную проверку скриптов + в течение дня при проверке фильтров, то в окне они нужны.

northis commented 2 years ago

@GdeZsqyAY так вручную их тоже можно обновить. @zubrRB не всегда, но в нескольких последних версиях так

zubrRB commented 2 years ago

Если я сегодня запустил систему в 9:00, а завтра запущу в 8:00, то во сколько произойдёт автопроверка юзерскриптов?

northis commented 2 years ago

@zubrRB отсчёт идёт от запуска службы. Сегодня была проверка в 9-30, если старт службы был в 9-00. Если вы выключите компьютер, а потом включите 8-00, то проверка будет в 8-30.

zubrRB commented 2 years ago

@northis где-то примерно в течение текущих суток до 00:00?

northis commented 2 years ago

@zubrRB Вы запустили сегодня в N часов, проверка будет в N+30 минут, а потом - раз в 24 часа, то есть в N+30 минут через сутки

zubrRB commented 2 years ago

@northis у меня система утром включается, на ночь выключается. Сегодня запустил в 10:00 (проверка в 10:30), а завтра в 00:01 ночи... Во сколько в этом случае проверка будет?

northis commented 2 years ago

@zubrRB в 10-30 сегодня и в 10-30 завтра

zubrRB commented 2 years ago

@northis и послезавтра, и пocлeпocлeзaвтpa... А система ведь каждый день запускается в разное время: раньше, позже.

northis commented 2 years ago

@zubrRB нет, я имел ввиду если не выключать компьютер. Само собой, если вы выключите, то отсчет начнётся заново.

zubrRB commented 2 years ago

@northis вот теперь понятно. А стандартная перезагрузка считается как новый запуск (с поверкой через 30 минут)?

zubrRB commented 2 years ago

30 минут с момента запуска, далее раз в 24 часа

@northis решил протестировать: юзерскрипт Handy Image 2021.12.17 обновился на github >14 часов назад, но у меня он почему-то автоматически не обновился ни при запуске, ни через 30 минут после запуска, ни через час после запуска...

Handy Image

northis commented 2 years ago

@zubrRB Пришлите логи. Лучше, конечно, расширенные. А вручную он обновляется?

zubrRB commented 2 years ago

@northis отправил логи (стандартные). А расширенные — это "подробный" уровень? Сейчас уже, видимо, нет смысла переключать на него, первая автопроверка ведь прошла. Вручную он и другие всегда обновляются при смене версии.

northis commented 2 years ago

@zubrRB Похоже, в метаданных скрипта есть ошибки https://uploads.adguard.com/win-northmtfgs.png

zubrRB commented 2 years ago

Установка через эту ссылку; https://github.com/Owyn/HandyImage/raw/master/HandyImage.user.js

northis commented 2 years ago
INFO, AdguardSvc, 13, 17.12.2021 10:34:03.977, Found 0 userscripts ready for update check
INFO, AdguardSvc, 13, 17.12.2021 10:34:03.977, 0 userscripts were updated

Проверка была, и в этот момент он должен был обновиться. Поисследуем, спасибо.

zubrRB commented 2 years ago

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

zubrRB commented 2 years ago
INFO, AdguardSvc, 80, 18.12.2021 10:36:30.180, Found 8 userscripts ready for update check
INFO, AdguardSvc, 80, 18.12.2021 10:37:09.618, 2 userscripts were updated

Сегодня с заданным подробным логированием обновился тот и дополнительно SaveFrom.net helper со своего сервера.

zubrRB commented 2 years ago
INFO, AdguardSvc, 9, 19.12.2021 11:34:46.142, Found 0 userscripts ready for update check
INFO, AdguardSvc, 9, 19.12.2021 11:34:46.149, 0 userscripts were updated

Спустя ещё сутки снова всё по нулям (в обеих строках 0). И в обоих этих случаях на момент автопроверки юзерскриптов был подключен VPN, а в день между ними, когда скрипты успешно автоматически обновились — VPN был отключен.

zubrRB commented 2 years ago
INFO, AdguardSvc, 57, 20.12.2021 10:41:36.978, Found 0 userscripts ready for update check
INFO, AdguardSvc, 57, 20.12.2021 10:41:36.982, 0 userscripts were updated

Опять по нулям с подробным логированием и включенным VPN.

zubrRB commented 2 years ago
INFO, AdguardSvc, 12, 21.12.2021 8:56:47.056, Found 0 userscripts ready for update check
INFO, AdguardSvc, 12, 21.12.2021 8:56:47.058, 0 userscripts were updated

Без использования VPN то же самое...

northis commented 2 years ago

@zubrRB А есть ли обновления для скриптов? Тот, который вы указали, пока не менялся - там версия 2021.12.17

zubrRB commented 2 years ago

При отсутствии обновлений меня смущает первая строка, которая видит 0 установленных скриптов в AdGuard. Ведь по логике, если приложение по некой причине не видит скриптов, готовых к проверке, то и обновить их не сможет.

northis commented 2 years ago

@zubrRB при ручной проверке все доступные скрипты проверяются на предмет обновлений. При автоматической - только те, у которых дата последнего обновления старше суток. Поэтому "некоторая причина" - это то, насколько давно обновлялся данный скрипт.

zubrRB commented 2 years ago

Дата последнего обновления или проверки обновлений? ПК у меня выключается ночью и свыше 24 часов не работает. Kinozal.tv magnet links 0.2 официально обновлялся год назад, не считая его псевдообновлений в программе. Собственные предустановленные юзерскрипты тоже обновлялись на прошлой неделе вместе с обновлением AdGuard. И единственные раз, когда в первой строке было правильно указано общее количество установленных скриптов (8), это когда два из них имели обновления и были обновлены без использования VPN и с подробным логированием. При этом один из них был доступен к обновлению за день до этого. Все остальные разы у меня обе строки изо дня в день стабильно выдают — 0.

zubrRB commented 2 years ago

@northis то всё по нулям, то якобы обновляет сразу все скрипты (что явно не так), то часть, то снова видит-не видит.

INFO, AdguardSvc, 48, 26.12.2021 9:28:06.619, Found 0 userscripts ready for update check
INFO, AdguardSvc, 48, 26.12.2021 9:28:06.630, 0 userscripts were updated

INFO, AdguardSvc, 13, 27.12.2021 9:52:39.104, Found 8 userscripts ready for update check
INFO, AdguardSvc, 13, 27.12.2021 9:53:12.689, 8 userscripts were updated

INFO, AdguardSvc, 9, 27.12.2021 22:20:17.999, Found 8 userscripts ready for update check
INFO, AdguardSvc, 9, 27.12.2021 22:21:24.017, 2 userscripts were updated

INFO, AdguardSvc, 34, 28.12.2021 9:37:45.751, Found 0 userscripts ready for update check
INFO, AdguardSvc, 34, 28.12.2021 9:37:45.765, 0 userscripts were updated

INFO, AdguardSvc, 39, 29.12.2021 10:37:14.235, Found 8 userscripts ready for update check
INFO, AdguardSvc, 39, 29.12.2021 10:37:20.147, 0 userscripts were updated
northis commented 2 years ago

@zubrRB

что явно не так

Почему не так?

zubrRB commented 2 years ago

Потому что по скриншоту расширений видно, что никакого полного их обновления утром 27.12 в реальности не было.

Screenshot: ![-](https://user-images.githubusercontent.com/50718960/147645453-0c425bac-e8bf-4b6e-a94a-5f34856e6ba6.png)

zubrRB commented 2 years ago

@northis эта проблема с двумя нолями возникает в случае, если новый запуск ОС происходит менее чем через 24 часа после предыдущей автопроверки. Если завтра я запущу систему на несколько минут раньше, чем сегодня, то будут два 0.
Хотя вы выше и указали, что после выключения ПК отсчёт начнётся заново, но в реальности этого не происходит.

INFO, AdguardSvc, 40, 03.01.2022 10:35:02.464, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 40, 03.01.2022 10:35:14.278, 0 userscripts were updated

INFO, AdguardSvc, 8, 04.01.2022 10:43:27.837, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 8, 04.01.2022 10:43:45.050, 0 userscripts were updated

INFO, AdguardSvc, 45, 05.01.2022 10:34:18.566, Found 0 userscripts ready for update check
INFO, AdguardSvc, 45, 05.01.2022 10:34:18.567, 0 userscripts were updated

INFO, AdguardSvc, 28, 06.01.2022 10:33:39.284, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 28, 06.01.2022 10:33:52.653, 0 userscripts were updated

INFO, AdguardSvc, 15, 06.01.2022 18:12:45.867, Found 0 userscripts ready for update check
INFO, AdguardSvc, 15, 06.01.2022 18:12:45.867, 0 userscripts were updated

INFO, AdguardSvc, 10, 07.01.2022 8:53:43.581, Found 0 userscripts ready for update check
INFO, AdguardSvc, 10, 07.01.2022 8:53:43.581, 0 userscripts were updated

INFO, AdguardSvc, 77, 08.01.2022 10:11:15.379, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 77, 08.01.2022 10:11:25.032, 0 userscripts were updated

INFO, AdguardSvc, 16, 08.01.2022 17:13:32.182, Found 0 userscripts ready for update check
INFO, AdguardSvc, 16, 08.01.2022 17:13:32.183, 0 userscripts were updated

INFO, AdguardSvc, 48, 08.01.2022 19:16:50.780, Found 0 userscripts ready for update check
INFO, AdguardSvc, 48, 08.01.2022 19:16:50.780, 0 userscripts were updated

INFO, AdguardSvc, 10, 09.01.2022 10:04:40.927, Found 0 userscripts ready for update check
INFO, AdguardSvc, 10, 09.01.2022 10:04:40.928, 0 userscripts were updated

INFO, AdguardSvc, 57, 10.01.2022 10:04:01.484, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 57, 10.01.2022 10:04:42.015, 0 userscripts were updated
zubrRB commented 2 years ago

@northis сегодня запустил систему на пять минут позже вчерашнего и поэтому снова произошла автопроверка UserJS.

INFO, AdguardSvc, 111, 11.01.2022 10:09:34.028, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 111, 11.01.2022 10:09:41.315, Userscript AdGuard Extra Beta has been updated to version 1.0.350
...
INFO, AdguardSvc, 111, 11.01.2022 10:09:43.121, 1 userscripts were updated
zubrRB commented 2 years ago

@northis сегодня ещё на полчаса позже и снова случилась автоматическая проверка скриптов с обновлением одного. Так что нужно, чтобы 24-часовой цикл автопроверки всё-таки прерывался при завершении работы и новом запуске ОС.

INFO, AdguardSvc, 34, 12.01.2022 10:34:59.690, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 34, 12.01.2022 10:35:07.207, Userscript AdGuard Extra Beta has been updated to version 1.0.351
...
INFO, AdguardSvc, 34, 12.01.2022 10:35:08.811, 1 userscripts were updated
northis commented 2 years ago

@zubrRB Спасибо за логи, разберёмся

zubrRB commented 2 years ago

@northis Ещё ручная проверка обновления фильтров и расширений сдвигает 24-часовой цикл автопроверки скриптов. В первый день утром была осуществлена автопроверка и дополнительно в течение дня задавались ручные проверки. В результате этого при более позднем запуске системы утром следующего дня автопроверка скриптов не произошла.

INFO, AdguardSvc, 10, 14.01.2022 10:07:54.711, Start checking userscripts updates. Forced=False
INFO, AdguardSvc, 10, 14.01.2022 10:07:57.564, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 10, 14.01.2022 10:08:06.714, 0 userscripts were updated

INFO, AdguardSvc, 11, 14.01.2022 18:16:04.733, Start checking userscripts updates. Forced=True
INFO, AdguardSvc, 11, 14.01.2022 18:16:06.557, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 11, 14.01.2022 18:16:16.813, Userscript AdGuard Extra Beta has been updated to version 1.0.352
...
INFO, AdguardSvc, 11, 14.01.2022 18:16:18.705, 1 userscripts were updated

INFO, AdguardSvc, 35, 14.01.2022 18:17:55.100, Start checking userscripts updates. Forced=True
INFO, AdguardSvc, 35, 14.01.2022 18:17:55.527, Found 8 userscripts ready for update check
INFO, AdguardSvc, 35, 14.01.2022 18:18:07.904, 0 userscripts were updated

INFO, AdguardSvc, 33, 14.01.2022 18:35:29.441, Start checking userscripts updates. Forced=True
INFO, AdguardSvc, 33, 14.01.2022 18:35:29.788, Found 8 userscripts ready for update check
INFO, AdguardSvc, 33, 14.01.2022 18:35:36.916, 0 userscripts were updated

INFO, AdguardSvc, 70, 14.01.2022 20:35:10.831, Start checking userscripts updates. Forced=True
INFO, AdguardSvc, 70, 14.01.2022 20:35:11.008, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 70, 14.01.2022 20:35:20.962, Userscript AdGuard Extra Beta has been updated to version 1.0.353
...
INFO, AdguardSvc, 70, 14.01.2022 20:35:23.059, 1 userscripts were updated
______
INFO, AdguardSvc, 9, 15.01.2022 10:40:00.476, Start checking userscripts updates. Forced=False
INFO, AdguardSvc, 9, 15.01.2022 10:40:14.334, Found 0 userscripts ready for update check
INFO, AdguardSvc, 9, 15.01.2022 10:40:14.334, 0 userscripts were updated
______
INFO, AdguardSvc, 12, 16.01.2022 10:34:39.678, Start checking userscripts updates. Forced=False
INFO, AdguardSvc, 12, 16.01.2022 10:34:46.780, Found 8 userscripts ready for update check
INFO, AdguardSvc, 12, 16.01.2022 10:35:03.894, 0 userscripts were updated
stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

zubrRB commented 2 years ago
INFO, AdguardSvc, 34, 13.08.2022 7:15:47.539, Start checking userscripts updates. Forced=False (при запуске ОС)
INFO, AdguardSvc, 34, 13.08.2022 7:15:51.207, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 34, 13.08.2022 7:15:57.832, 0 userscripts were updated

INFO, AdguardSvc, 38, 13.08.2022 17:00:37.916, Start checking userscripts updates. Forced=True (ручная проверка)
INFO, AdguardSvc, 38, 13.08.2022 17:00:38.538, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 38, 13.08.2022 17:00:48.954, 1 userscripts were updated

INFO, AdguardSvc, 15, 14.08.2022 0:44:57.010, Start checking userscripts updates. Forced=True (ручная проверка)
INFO, AdguardSvc, 15, 14.08.2022 0:45:00.523, Found 8 userscripts ready for update check
...
INFO, AdguardSvc, 15, 14.08.2022 0:45:11.247, 0 userscripts were updated

INFO, AdguardSvc, 19, 14.08.2022 7:34:39.672, Start checking userscripts updates. Forced=False (при запуске ОС)
INFO, AdguardSvc, 19, 14.08.2022 7:34:42.146, Found 0 userscripts ready for update check
INFO, AdguardSvc, 19, 14.08.2022 7:34:42.146, 0 userscripts were updated
northis commented 1 year ago

@zubrRB

эта проблема с двумя нолями возникает в случае, если новый запуск ОС происходит менее чем через 24 часа после предыдущей автопроверки. 

Запуск ОС полностью сбрасывает таймер расписаний для скриптов и фильтров. Отсчет всегда ведётся от начала запуска службы. Данные об этом хранятся в памяти и само собой обнуляются при перезагрузке/выключении компьютера. Другое дело, когда компьютер уходи в сон, тогда формально цикл выполнения расписаний не прерывается.

По обновлению скриптов - поменяем логику, должно помочь.

zubrRB commented 1 year ago

Если я завтра запущу систему на секунду раньше, чем сегодня, то автоматической проверки скриптов не произойдёт.

northis commented 1 year ago

@zubrRB вы полностью выключаете компьютер или он уходит в сон? Проблема 8 userscripts were updated, когда на скрине не видно того, что 8 скриптов обновлено, также связана с тем, что в этот момент нет интернета (скорее всего, при выходе из сна), и мы ошибочно помечаем в логе все эти скрипты как обновлённые (исправим в найтли 7.12)

zubrRB commented 1 year ago

Да, на ночь полностью выключается через "Завершение работы" с полным отключением от сети электропитания. 24-часовой цикл жёстко соблюдается и никак не сбивается, в том числе ручная проверка обновлений сдвигает его.

northis commented 1 year ago

@zubrRB всё же нужны будут расширенные логи, пока не очень понятно, что вы имеете ввиду под "сдвигом обновления". Также хочу отметить, что обновление скриптов происходит первый раз спустя 30 минут после запуска службы.

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

Насколько я вижу, автоматические проверки в следующие дни таки были

______
INFO, AdguardSvc, 9, 15.01.2022 10:40:00.476, Start checking userscripts updates. Forced=False
INFO, AdguardSvc, 9, 15.01.2022 10:40:14.334, Found 0 userscripts ready for update check
INFO, AdguardSvc, 9, 15.01.2022 10:40:14.334, 0 userscripts were updated
______
INFO, AdguardSvc, 12, 16.01.2022 10:34:39.678, Start checking userscripts updates. Forced=False
INFO, AdguardSvc, 12, 16.01.2022 10:34:46.780, Found 8 userscripts ready for update check
INFO, AdguardSvc, 12, 16.01.2022 10:35:03.894, 0 userscripts were updated
zubrRB commented 1 year ago

Сдвиг 24-часового цикла автоматической проверки обновлений хорошо виден на приведённом вами примере... Вечером 14-го числа была задана ручная проверка, а 15-го утром автоматической не было, но она была утром 16-го.

northis commented 1 year ago

15-го утром автоматической не было

INFO, AdguardSvc, 9, 15.01.2022 10:40:00.476, Start checking userscripts updates. Forced=False

Это автоматическая проверка