Skip to content

fix(tunnel): cortar cada verificación a los 5 s y dar 60 s en total - #2

Merged
KonixDev merged 1 commit into
mainfrom
fix/tunnel-timeout
Sep 30, 2026
Merged

KonixDev merged 1 commit into
mainfrom
fix/tunnel-timeout

Conversation

@KonixDev

Copy link
Copy Markdown
Owner

Problema

En una Mac, "Compartir por internet" se quedaba en Conectando… y terminaba con:

la dirección https://….trycloudflare.com no respondió: Get ".../healthz": context deadline exceeded

Cada verificación de /healthz podía esperar hasta 10 s, y el plazo total era de 30 s. Un intento colgado mientras la dirección nueva se propaga se comía un tercio del plazo.

Cambio

  • verifyClient: cada intento corta a los 5 s (antes 10 s), así se reintenta rápido.
  • Start: plazo por defecto de 60 s (antes 30 s).

Verificación

  • Test nuevo TestVerifyHealthzDoesNotHang: fallaba antes (10 s) y ahora pasa.
  • go test ./... en verde.
  • Con cloudflared real, 3 intentos: conectó en 11 s, 24 s y 23 s. Dos de los tres quedaron cerca del límite viejo.

🤖 Generated with Claude Code

Una verificación de /healthz colgada podía comerse 10 s de los 30 del
plazo y dejar el túnel en "Conectando…" hasta fallar con
"context deadline exceeded". En pruebas reales el túnel tardó 11–24 s
en responder, muy cerca del límite viejo.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@KonixDev
KonixDev merged commit b72bac6 into main Sep 30, 2026
11 checks passed
@KonixDev
KonixDev deleted the fix/tunnel-timeout branch September 30, 2026 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant