Skip to content

Conversation

@Utkarsh9571
Copy link

Rebased and cherry-picked to show only relevant commits as requested. Includes SSH tunneling logic, config patch, and contributor-safe test coverage.

@Utkarsh9571
Copy link
Author

@vaibhav-datazip hey man, is the solution good.

@vaibhav-datazip
Copy link
Collaborator

hi @Utkarsh9571 , was busy in some other features. Will be reviewing your PR soon.
to make the review fast can you please list the steps how you have tested your PR on both the drivers. and also share a video of the same.

@Utkarsh9571
Copy link
Author

@vaibhav-datazip Thanks! I’ve tested the patch on both drivers as follows:

✅ OpenSSH Driver

  • Started tunnel with datazip tunnel start --driver openssh
  • Simulated silent channel close by killing the remote process
  • Verified restart logic via logs: watch loop restarted and tunnel reconnected

✅ Cloudflared Driver

  • Started tunnel with datazip tunnel start --driver cloudflared
  • Simulated disconnect by blocking outbound connection
  • Verified retry logic and reconnection via logs

Let me know if you'd like me to add anything else!

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.

2 participants