Flush the connection log buffer after N failed reconnect attempts against an unreachable server
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- networking, observability
Hướng nghiên cứu
Start by reading src/websocket/reconnectingWebSocket.ts and src/logging/logBuffer.ts, then review PR #1100 for the existing flush triggers and reasons. Settle the retry threshold, counter reset, and one-flush-per-failure-episode behavior. Done means sustained unreachable-server retries flush the buffer once, a successful connection resets the episode, and later terminal failures retain their existing behavior.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Background
DEVEX-669 (#1100) added BufferingLogger, which flushes buffered below-level connection logs into the "Coder" output channel on a genuine, surfaced connection failure:
- a reconnecting WebSocket terminal failure (
unrecoverable_close,unrecoverable_http, orcertificate_error; a handshake401is excluded because a token refresh reconnects the same socket); - a failure while opening a workspace (canceled build, missing agent, timeout, or CLI/certificate error).
During Ehab's review of #1100 we agreed to defer one additional trigger: flushing after N consecutive failed reconnect attempts against an unreachable server.
Problem
When a server is simply unreachable, the reconnecting WebSocket keeps retrying with backoff and never reaches a terminal failure reason, so the buffer is never flushed. A user reproducing a "can't connect / hangs on connecting" problem therefore gets no buffered detail in their support bundle, even though plenty of below-level context was captured.
Proposal
Flush the connection log buffer once a reconnect loop has failed N consecutive times (i.e., treat sustained unreachability as a genuine failure worth surfacing), then avoid re-flushing on every subsequent attempt until a successful connect resets the counter.
Open questions to settle during implementation:
- Value of N (or a time-based threshold instead of / in addition to a count).
- Where the counter lives and how it resets on a successful open.
- Guarding against repeated flushes while still stuck (flush once per failure "episode").
- A meaningful flush
reason/route for attribution, consistent with the existing terminal-failure reasons.
References
- PR #1100 (DEVEX-669)
src/websocket/reconnectingWebSocket.ts,src/logging/logBuffer.ts
🤖 Filed by Coder Agents on behalf of @aqandrew, following up on the #1100 review.
- Ngôn ngữ chính
- TypeScript
- Star
- 131
- Fork
- 48
- Merge trung bình
- 2 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 16
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của coder/vscode-coder
-
Split commands.ts by topicĐang mởImprovement tech-debt vs-code
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
coder/vscode-coder#1135 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Migrate CI setup to pnpm/setupCó thể đã có người làm @EhabY đã nhận 17 ngày trước. Đang mởtech-debt
coder/vscode-coder#1119 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Design the VS Code Workspaces panelCó thể đã có người làm @chrifro đã nhận 21 ngày trước. Đang mở
coder/vscode-coder#1113 · 3 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
-
bug upstream
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
coder/vscode-coder#1087 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Improvement tech-debt vs-code
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
coder/vscode-coder#1066 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của coder/vscode-coder
Issue tương tự
-
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 85/100
capricorn86/happy-dom#2474 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
JSerwatka/letterboxd-tweaks#81 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
siyuan-note/siyuan#20165 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
polkadot-js/phishing#5716 ·
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
juice-shop/juice-shop#3662 ·
Maintainer thường phản hồi trong vòng 1 ngày