fix: translate permanent invalid-token errors on online entrypoints (WealthLedger/backend#59) #110

Merged

codex-bot merged 1 commits from fix/59-online-invalid-token-translation into develop 2026-10-16 23:23:12 +03:00

Conversation 0 Commits 1 Files Changed 8

+240 -64

codex-bot commented 2026-10-16 22:54:02 +03:00

Owner

Copy Link

Copy Source

Refs WealthLedger/backend#59

Что

Первый upstream UNAUTHENTICATED/PERMISSION_DENIED на online search/detail/ticker/company/screener теперь возвращает 400 / INVALID_TBANK_TOKEN вместо 500. Временная недоступность остаётся 503 / TBANK_UNAVAILABLE.

TbankAccessService.onlineToken и current заменены одним online(userId, run): он берёт токен и выполняет run внутри существующего translateInvalidToken. Токен вне замыкания получить нельзя, поэтому новый online-вызов не сможет обойти перевод. Новых классов ошибок и зависимостей нет.

Не затронуто

• Durable-ветки (tokenForAttempt, rejectMembershipToken, их catches) и attempt-aware invalidation.
• Current-revision isolation: revision по-прежнему приходит из того же чтения connection.
• Online-отказ ничего не пишет в БД: invalidToken и revision остаются прежними.

Тесты

Новый describe в test/stocks.e2e-spec.ts и test/bonds.e2e-spec.ts (хелпер test/online-upstream.ts). Реальные сервисы и SQLite, подменён только gRPC-транспорт, поэтому статус проходит через настоящий маппинг TbankClient.

• Share: 5 entrypoints x {UNAUTHENTICATED, PERMISSION_DENIED} x {первый RPC, поздний RPC} + UNAVAILABLE.
• Bond: search/detail/ticker (company и screener у bond нет).
• Тело ответа строго {code, status}: без raw-текста ошибки и токена; invalidToken=false, revision не изменился.

Мутации: без translateInvalidToken в online падают 20 из 48 кейсов stocks; обход перевода в screener роняет ровно 4 screener-кейса.

Gates

• npm run verify: exit 0, 40 suites, 288 tests.
• npm run test:e2e: exit 0, все 22 suites PASS.
• openapi:check не запускался: wire contract и Swagger не менялись.

Consumer proof (frontend GET не ретраит 400/INVALID_TBANK_TOKEN) вынесен в отдельный frontend PR.

Refs WealthLedger/backend#59 ## Что Первый upstream `UNAUTHENTICATED`/`PERMISSION_DENIED` на online search/detail/ticker/company/screener теперь возвращает `400 / INVALID_TBANK_TOKEN` вместо `500`. Временная недоступность остаётся `503 / TBANK_UNAVAILABLE`. `TbankAccessService.onlineToken` и `current` заменены одним `online(userId, run)`: он берёт токен и выполняет `run` внутри существующего `translateInvalidToken`. Токен вне замыкания получить нельзя, поэтому новый online-вызов не сможет обойти перевод. Новых классов ошибок и зависимостей нет. ## Не затронуто - Durable-ветки (`tokenForAttempt`, `rejectMembershipToken`, их catches) и attempt-aware invalidation. - Current-revision isolation: `revision` по-прежнему приходит из того же чтения connection. - Online-отказ ничего не пишет в БД: `invalidToken` и `revision` остаются прежними. ## Тесты Новый `describe` в `test/stocks.e2e-spec.ts` и `test/bonds.e2e-spec.ts` (хелпер `test/online-upstream.ts`). Реальные сервисы и SQLite, подменён только gRPC-транспорт, поэтому статус проходит через настоящий маппинг `TbankClient`. - Share: 5 entrypoints x {UNAUTHENTICATED, PERMISSION_DENIED} x {первый RPC, поздний RPC} + UNAVAILABLE. - Bond: search/detail/ticker (company и screener у bond нет). - Тело ответа строго `{code, status}`: без raw-текста ошибки и токена; `invalidToken=false`, `revision` не изменился. Мутации: без `translateInvalidToken` в `online` падают 20 из 48 кейсов stocks; обход перевода в screener роняет ровно 4 screener-кейса. ## Gates - `npm run verify`: exit 0, 40 suites, 288 tests. - `npm run test:e2e`: exit 0, все 22 suites PASS. - `openapi:check` не запускался: wire contract и Swagger не менялись. Consumer proof (frontend GET не ретраит 400/`INVALID_TBANK_TOKEN`) вынесен в отдельный frontend PR.

codex-bot added 1 commit 2026-10-16 22:54:03 +03:00

fix: translate permanent invalid-token errors on online entrypoints (WealthLedger/backend#59) c18e2b9d92

codex-bot merged commit 8a6f60d965 into develop 2026-10-16 23:23:12 +03:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Kind/Dependencies

Dependency update or dependency-management work

Kind/Investigation

Research or decision required before implementation

Kind/Process

Developer workflow, CI/CD, tooling, or process improvement

Kind/Refactor

Internal code or configuration restructuring without new product behavior

audit/2026-10-11

Technical Audit Remediation — 2026-10-11

kind/investigation

Нужна проверка evidence до решения; runtime-дефект не утверждается

Compat/Breaking

Breaking change that won't be backward compatible

Kind/Bug

Something is not working

Kind/Documentation

Documentation changes

Kind/Enhancement

Improve existing functionality

Kind/Feature

New functionality

Kind/Security

This is security issue

Kind/Testing

Issue or pull request related to testing

Priority

Critical

1

The priority is critical

Priority

High

2

The priority is high

Priority

Medium

3

The priority is medium

Priority

Low

4

The priority is low

Reviewed

Confirmed

1

Issue has been confirmed

Reviewed

Duplicate

2

This issue or pull request already exists

Reviewed

Invalid

3

Invalid issue

Reviewed

Won't Fix

3

This issue won't be fixed

Status

Blocked

1

Something is blocking this issue or pull request

Status

Need More Info

2

Feedback is required to reproduce issue or to continue work

Status

Abandoned

3

Somebody has started to work on this but abandoned work

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

codex-bot hermes-agent ksv741 publisher-bot renovate-bot

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: WealthLedger/backend#110