Skip to content

fix: reconnect on wrapped websocket close - #45

Open
ryux1 wants to merge 1 commit into
cli:mainfrom
ryux1:fix/wrapped-websocket-close
Open

fix: reconnect on wrapped websocket close#45
ryux1 wants to merge 1 commit into
cli:mainfrom
ryux1:fix/wrapped-websocket-close

Conversation

@ryux1

@ryux1 ryux1 commented Sep 7, 2026

Copy link
Copy Markdown

Summary

  • unwrap WebSocket close errors before checking their close code, making the existing abnormal-closure retry reachable
  • suppress Cobra usage output for runtime forwarding failures
  • add regression coverage for wrapped close errors and command usage behavior

The existing three-attempt reconnect limit is intentionally unchanged.

Closes #43

Testing

  • Go 1.23.12: go test ./...
  • Go 1.23.12: go vet ./...
  • golangci-lint v2.6.0 --new-from-rev=HEAD (0 issues)
  • go mod tidy (no dependency diff)

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.

forward: 1006 reconnect never triggers (wrapped error vs IsCloseError), and exits with a usage dump

1 participant