Follow-ups from #9 #12

Open
opened 2026-10-05 19:22:09 +03:00 by aep · 0 comments
Owner

Отложенные находки ревью технического долга Среза 1 (PR #10). Маркер — суть — откуда.

  • R-1 остаток — троттлинг/дедуп LOGIN_FAILED по (ip, причина) за окно или решение «режет шлюз платформы» — ревью п.5
  • R-5 — LOGIN первого запроса берёт второе соединение из пула; при всплеске после рестарта возможна потеря события (логируется) — ревью п.5
  • R-6 — LOGIN пишется при каждой выдаче dev-токена (dev-вход = два события) — ревью п.5
  • R-8 — 404 под /api/v1 и у статики, 405 от StaticFiles на POST / — английские тексты detail — ревью п.3
  • R-10 — create_app(settings): lifespan/S3/NATS читают модульный settings — ревью п.2
  • R-12 — CORS_ORIGINS без нормализации хвостового / — ревью п.2
  • R-13 — тесты: LOGIN_FAILED для неоднозначной учётной записи и «аутентификация не настроена»; create_app ставит SessionAudit; HEAD/OPTIONS под /api/v1 — ревью
  • OQ-62 остаток — фронт не собирается в образ arm-core — Срез 0
  • OQ-65 остаток — перечень доверенных прокси прода для FORWARDED_ALLOW_IPS — к платформе
  • CLAUDE.md, пункт «Аудит»: «в текущей транзакции» — уточнить исключением для SESSION — ревью п.5
  • OQ-59 — уникальность email без учёта регистра на уровне БД — DBML v5.15 (подсрез 2c)
Отложенные находки ревью технического долга Среза 1 (PR #10). Маркер — суть — откуда. - R-1 остаток — троттлинг/дедуп `LOGIN_FAILED` по (ip, причина) за окно или решение «режет шлюз платформы» — ревью п.5 - R-5 — `LOGIN` первого запроса берёт второе соединение из пула; при всплеске после рестарта возможна потеря события (логируется) — ревью п.5 - R-6 — `LOGIN` пишется при каждой выдаче dev-токена (dev-вход = два события) — ревью п.5 - R-8 — 404 под `/api/v1` и у статики, 405 от StaticFiles на `POST /` — английские тексты `detail` — ревью п.3 - R-10 — `create_app(settings)`: lifespan/S3/NATS читают модульный `settings` — ревью п.2 - R-12 — `CORS_ORIGINS` без нормализации хвостового `/` — ревью п.2 - R-13 — тесты: `LOGIN_FAILED` для неоднозначной учётной записи и «аутентификация не настроена»; `create_app` ставит `SessionAudit`; HEAD/OPTIONS под `/api/v1` — ревью - OQ-62 остаток — фронт не собирается в образ `arm-core` — Срез 0 - OQ-65 остаток — перечень доверенных прокси прода для `FORWARDED_ALLOW_IPS` — к платформе - CLAUDE.md, пункт «Аудит»: «в текущей транзакции» — уточнить исключением для SESSION — ревью п.5 - OQ-59 — уникальность email без учёта регистра на уровне БД — DBML v5.15 (подсрез 2c)
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RSHB/arm-core#12
No description provided.