Skip to content

Минимальные разрешения: без tabs, доступ только к http/https - #4

Merged
goodprogrammist merged 1 commit into
masterfrom
claude/festive-bardeen-b8f0m7
Oct 11, 2026
Merged

goodprogrammist merged 1 commit into
masterfrom
claude/festive-bardeen-b8f0m7

Conversation

@goodprogrammist

Copy link
Copy Markdown
Owner

Что сделано

Перед публикацией в магазинах сокращены разрешения: магазины требуют запрашивать только необходимое.

  • Убрано разрешение tabs. По MDN (tabs.Tab.url) URL вкладки доступен и при подходящем разрешении на сайты, поэтому tabs не нужен.
  • <all_urls> сужен до http://*/* и https://*/*: cookie Xdebug имеют смысл только там.
  • Проверка прав в common.js (permissions.contains / permissions.request) использует те же шаблоны.
  • Тест Chromium больше не ищет вкладку about:blank по URL: без tabs такие вкладки по URL не находятся. Расширение на этой странице по-прежнему показывает «недоступно».
  • store/permissions.md обновлён.

Проверка

  • Chromium 141: 13/13, 3 прогона подряд.
  • Firefox 157.0.1 и 153.4.0esr: 9/9.
  • web-ext lint: 0 ошибок, 1 ожидаемое предупреждение.

CI не запускается из-за блокировки аккаунта по оплате (см. предыдущие PR).

🤖 Generated with Claude Code

https://claude.ai/code/session_011ZZMU6eeKUpne5HfQBAxoT


Generated by Claude Code

- tab.url is available with matching host permissions, "tabs" is not needed
- cookies make sense only for http(s), <all_urls> narrowed to http://*/* and https://*/*
- permission check in common.js uses the same origins
- Chromium test no longer looks up about:blank tab by URL
- store/permissions.md updated

Passed locally: Chromium 141 13/13 (3 runs), Firefox 157.0.1 and 153.4.0esr 9/9.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011ZZMU6eeKUpne5HfQBAxoT
@goodprogrammist
goodprogrammist merged commit 0155158 into master Oct 11, 2026
0 of 6 checks passed
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.

2 participants