Skip to content

Инструменты: fetch_assets качает через curl, если Python не доверяет сертификатам - #46

Merged
devAsmodeus merged 1 commit into
mainfrom
fetch-assets-ssl
Sep 29, 2026
Merged

devAsmodeus merged 1 commit into
mainfrom
fetch-assets-ssl

Conversation

@devAsmodeus

Copy link
Copy Markdown
Owner

Что было не так

Первая проверка tools/fetch_assets.py на настоящем релизе assets-v1.3.12 упала с CERTIFICATE_VERIFY_FAILED. У Python с python.org на macOS нет системных корневых сертификатов, пока не запущен Install Certificates.command, поэтому urllib не доверяет github.com.

Что сделано

  • При ошибке проверки сертификата _get скачивает файл через curl: он использует системные сертификаты (на macOS — связку ключей). Остальные ошибки, например 404, по-прежнему поднимаются как есть.
  • Тесты: запасной путь через curl и то, что другие ошибки не прячутся.

Как проверено

  • python3 tools/fetch_assets.py --only json на настоящем релизе скачал 4 028 файлов за 7 с. ui/, visuals/ и frames_index.json совпали с локальной сборкой байт в байт.
  • ruff check . проходит, все тесты зелёные.

🤖 Generated with Claude Code

…сертификатам

У Python с python.org на macOS нет системных корневых сертификатов (до
Install Certificates.command), и urllib падает с CERTIFICATE_VERIFY_FAILED
на github.com. При такой ошибке скачиваем через curl — он берёт системные
сертификаты; другие ошибки не прячем. Проверено на настоящем релизе
assets-v1.3.12: JSON-часть совпала с локальной сборкой.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@devAsmodeus
devAsmodeus merged commit cdfbab9 into main Sep 29, 2026
3 checks passed
@devAsmodeus
devAsmodeus deleted the fetch-assets-ssl branch September 29, 2026 12:34
devAsmodeus added a commit that referenced this pull request Sep 30, 2026
…сертификатам (#46)

При CERTIFICATE_VERIFY_FAILED (Python с python.org на macOS без Install Certificates.command) fetch_assets.py скачивает через curl с системными сертификатами; другие ошибки не прячет. Проверено на релизе assets-v1.3.12: обе части совпали с локальной сборкой байт в байт.
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.

1 participant