Decide whether tlsHandshakeTimeoutMs should cover the ClientHello peek
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 48/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ệ
- rust
- Lĩnh vực
- networking
Hướng nghiên cứu
Read the timeout handling in src/sni.rs, src/proxy_conn.rs, and src/failover.rs, along with the README’s description of tlsHandshakeTimeoutMs. Decide whether the setting covers only the TLS handshake or total connection setup; then either document that meaning or carry an accept-time deadline through the three paths, accounting for the ClientHello peek. Done means the timeout behavior and its documented meaning agree.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
tlsHandshakeTimeoutMs starts counting after the ClientHello peek, so a client's total wait from accept can exceed it by up to the peek's own 5 s limit.
Every user of the timeout starts from the same point, after sni::peek (REASSEMBLY_TIMEOUT in src/sni.rs):
- the TLS handshake:
timeout(hs_timeout, tls_accept(..))insrc/proxy_conn.rs - peer forwarding:
forward_to_peersinsrc/failover.rs - the passthrough local connect plus peer forward from Forward a connection to a peer when its local upstream connect fails (#79)
Nothing is wrong if the setting means "time for the handshake". If an operator reads it as a cap on total connection setup, it is not one. Decide which it means. Then either state it in the README, or start one deadline at accept and deduct the peek time in all three places.
Review-Deferral: 13616fe4f6c36296 severity=nit category=reliability
🤖 Generated by Claude Opus; posted via @kriszyp.
- Ngôn ngữ chính
- Rust
- Star
- 1
- Fork
- 0
- Merge trung bình
- 19 giờ 12 phút
- Pull request đã merge (30 ngày)
- 13
Chuẩn bị môi trường
- 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 HarperFast/symphony
-
Don't route to a restarted upstream until it serves, not just acceptsCó thể đã có người làm @kriszyp đã nhận 5 ngày trước. Đang mở
HarperFast/symphony#84 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Forward to a peer when the local upstream connect fails on a TLS-terminating failover routeCó thể đã có người làm @kriszyp đã nhận 5 ngày trước. Đang mở
HarperFast/symphony#83 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Forward a connection to a peer when its local upstream connect failsCó thể đã có người làm @kriszyp đã nhận 5 ngày trước. Đang mở
HarperFast/symphony#79 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
HarperFast/symphony#77 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Spill a share of new connections to a less-loaded peer when local Harper threads are busyCó thể đã có người làm @kriszyp đã nhận 12 ngày trước. Đang mở
HarperFast/symphony#73 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của HarperFast/symphony
Issue tương tự
-
area: cli kind:docs priority: P3
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
swc-project/swc-ai-issues#164 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
pop-os/cosmic-player#367 ·
-
good first issue help wanted
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
servo/rust-smallvec#873 ·