fix: separate T-Bank RPC network time from queue wait (WealthLedger/backend#81) #94

Merged

codex-bot merged 1 commits from 81-tbank-grpc-diagnostics into develop 2026-10-15 05:41:22 +03:00

Conversation 0 Commits 1 Files Changed 3

+9 -2

codex-bot commented 2026-10-14 23:04:07 +03:00

Owner

Copy Link

Copy Source

Closes #81 (частично: диагностика).

Что изменено

• Добавлено поле networkMs в RpcDiagnostics: время только сетевой попытки gRPC (от отправки RPC до ответа/ошибки), отдельно от queueWaitMs (ожидание локального лимитера).
• networkMs разрешён в allowlist структурного логгера.
• Тест расширен проверкой наличия networkMs в событии tbank.rpc.failed; тест падает без правки.

Не изменено

Контракт, deadline (15 с), retry policy, коды ошибок, HTTP-контракт.

Проверка

• npm run verify — exit 0, 268 тестов, 40 suites.
• Тест distinguishes a gRPC deadline from queue delay... падает при удалении networkMs.
• Токен и Authorization не попадают в логи (проверка JSON.stringify(calls)).

Остаётся открытым: причина самих timeouts (сетевая/канал) — определяется после накопления логов в реальном окружении.

Closes #81 (частично: диагностика). ## Что изменено - Добавлено поле `networkMs` в `RpcDiagnostics`: время только сетевой попытки gRPC (от отправки RPC до ответа/ошибки), отдельно от `queueWaitMs` (ожидание локального лимитера). - `networkMs` разрешён в allowlist структурного логгера. - Тест расширен проверкой наличия `networkMs` в событии `tbank.rpc.failed`; тест падает без правки. ## Не изменено Контракт, deadline (15 с), retry policy, коды ошибок, HTTP-контракт. ## Проверка - `npm run verify` — exit 0, 268 тестов, 40 suites. - Тест `distinguishes a gRPC deadline from queue delay...` падает при удалении `networkMs`. - Токен и `Authorization` не попадают в логи (проверка `JSON.stringify(calls)`). Остаётся открытым: причина самих timeouts (сетевая/канал) — определяется после накопления логов в реальном окружении.

codex-bot added 1 commit 2026-10-14 23:04:08 +03:00

fix: separate T-Bank RPC network time from queue wait (WealthLedger/backend#81) 2c24385311

codex-bot merged commit 0d7839cf79 into develop 2026-10-15 05:41:22 +03:00

codex-bot deleted branch 81-tbank-grpc-diagnostics 2026-10-15 05:41:22 +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#94