Skip to content

fix(dev): маршруты API приглашений и шаблонов решений - #835

Merged
TThanos3000 merged 1 commit into
devfrom
fix/dev-solution-api-proxy-20261026
Oct 26, 2026
Merged

TThanos3000 merged 1 commit into
devfrom
fix/dev-solution-api-proxy-20261026

Conversation

@TThanos3000

@TThanos3000 TThanos3000 commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Что изменено

DEV Nginx проксирует существующие C00 namespaces /solution-invitations/ и /solution-form-templates/ в Django. До исправления они попадали в Angular SPA:HTTP200 HTML, ошибка JSON parsing в уведомлениях. Путь решения внутри /programs/ уже работал.

API / UI

Контракты, Angular и бизнес-правила не менялись. Новые API paths получают ответы Django (включая401 JSON без авторизации), существующий allowlist auth SPA и остальные namespaces сохранены.

Permissions

Backend authentication/object permissions сохранены; новые regex alternatives имеют прежнюю границу /|$. Похожий SPA URL /solution-invitations-about остается SPA. Изменяется только DEV config, production не затрагивается.

Проверки

  • Reproduced in live browser: polling invitations returns200 HTML and parsing errors; exact missing namespaces found in config.
  • Flake8/diff check PASS; source review:3 files,13 additions,3 deletions; old routing rules preserved.
  • Real Nginx Linux CI40323667925 PASS: оба root API, вложенные paths, похожий SPA prefix и все прежние SPA/API paths.
  • Финальный deploy40324125323 на625517e6 PASS: оба root401 JSON, smoke API/SPA и worker ping. Прямой внешний HTTP повторно подтвердил401 JSON; свежая авторизованная browser-проверка уведомлений и переходов завершилась без новых parsing/console errors. Предыдущая сессия некоторое время сохраняла старый ответ; после повторного входа проверка пройдена.

Риски / migration

Миграций нет. В post-deploy routing smoke добавлены обе проверки401 JSON, чтобы повторная потеря proxy namespace останавливала выкладку. Изменений старых программ, проектов или данных нет.

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