Decide whether tlsHandshakeTimeoutMs should cover the ClientHello peek
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- rust
- Ambito
- networking
Direzione di ricerca
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.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Rust
- Stelle
- 1
- Fork
- 0
- Merge medio
- 19h 12m
- PR unite (30g)
- 13
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di HarperFast/symphony
-
Don't route to a restarted upstream until it serves, not just acceptsForse già presa @kriszyp l’ha presa 5 giorni fa. Aperta
HarperFast/symphony#84 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Forward to a peer when the local upstream connect fails on a TLS-terminating failover routeForse già presa @kriszyp l’ha presa 5 giorni fa. Aperta
HarperFast/symphony#83 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Forward a connection to a peer when its local upstream connect failsForse già presa @kriszyp l’ha presa 5 giorni fa. Aperta
HarperFast/symphony#79 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
HarperFast/symphony#77 ·
I maintainer di solito rispondono entro 1 giorno
-
Spill a share of new connections to a less-loaded peer when local Harper threads are busyForse già presa @kriszyp l’ha presa 12 giorni fa. Aperta
HarperFast/symphony#73 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di HarperFast/symphony
Issue simili
-
area: cli kind:docs priority: P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
swc-project/swc-ai-issues#164 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
pop-os/cosmic-player#367 ·
-
good first issue help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
servo/rust-smallvec#873 ·