Repository navigation
Tích hợp AccountPool vào proxy handler (failover-ready routing) #4
Description
Tích hợp AccountPool vào handler
What to build
Nối AccountPool vào request path của proxy, giữ behavior cũ cho 1 acc:
- Handler pick acc qua
pool.reserve()thay vìauth.getCredential()trực tiếp;finally+res.on('close')release — mọi đường thoát: stream xong, lỗi upstream, client abort, response hoàn thành bình thường. Client cancel KHÔNG ảnh hưởng health acc. - Header debug
x-zcode-account: <id>chỉ khidebug: true(không leak acc cho client ngoài). - 1 acc duy nhất → behavior y như cũ: không thêm queue, không thêm header trừ khi debug, tests cũ của handler phải pass không sửa (trừ chỗ credential inject).
- Pool wiring: AuthManager giữ vai trò trước store cho compat; handler đọc qua pool.
- Nền cho failover (ticket 3): pick → reserve → inject credential → forward. Chưa có retry sang acc khác trong ticket này.
Acceptance criteria
- 2 acc healthy → 2 request liên tiếp dùng 2 acc khác nhau (xác nhận qua header debug)
- 1 acc → behavior cũ: tests handler hiện tại pass, không regression
- Release chạy trên mọi đường thoát: test stream xong / upstream lỗi / client abort giữa SSE / client abort trước headers — inFlight về 0 sau mỗi case
- Client abort không làm acc cooling
- Header
x-zcode-accountchỉ xuất hiện khi debug bật - AuthManager compat: các consumers cũ (quota, TUI, Android, claim) vẫn chạy được
Activity
longphamdev commentedon Nov 9, 2026
Bổ sung acceptance (từ review #5):
longphamdev commentedon Nov 9, 2026
Prerequisite từ review #1 (bắt buộc trước khi nối pool):
2 test
sendOrderedUpstreamRequestabort đang fail sẵn (đã verify pre-existing tại #1). Nhưngreserve/releaseabort-safe (#3) và test release-khi-client-abort ở ticket này dựa vào đúng hành vi abort củasendOrderedUpstreamRequest— nền hiện tại không tin cậy.Bước đầu của ticket này: điều tra 2 test fail đó — sửa test (nếu flaky timing) hoặc sửa code (nếu bug abort propagation thật) — trước khi viết bất kỳ logic pool-integration nào. Chấp nhận: test abort pass ổn định hoặc có phân tích rõ ràng vì sao hành vi hiện tại vẫn an toàn cho pool.
longphamdev commentedon Nov 9, 2026
Tích hợp AccountPool hoàn thành.
Prereq (abort tests): root cause là Bun 1.3.x regression — server-side http socket không phát close event khi client destroy đột ngột (verify bằng raw net.Socket trên Node 23: cùng code phát close đúng). Code proxy đúng (promise reject + destroy client-side + postWrite flag). Fix: tests pin contract có thể quan sát (reject + đúng 1 upstream dispatch, không retry); comment ghi rõ để re-assert khi Bun sửa.
Integration:
credchung)x-zcode-accountchỉ khi debug: trueSuite: 1181 pass / 0 fail (baseline 1140/2). Frontier: #5 và #6 (song song).
feat(proxy): integrate AccountPool into request path (#4)
fix(proxy): watchdog aborts upstream + stream-wrapper release (#5 pre…