Skip to content

Support connections over SSH / drivers #1470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 19, 2025

Conversation

d-mato
Copy link
Contributor

@d-mato d-mato commented Apr 14, 2025

SSH Tunneling Implementation for Individual Database Drivers

This pull request implements SSH tunneling support for individual database drivers, building upon the core SSH tunneling added in the previous PR (#1447).

Note: This PR will be rebased once the core PR is merged to ensure a clean commit history.

@d-mato d-mato marked this pull request as draft April 14, 2025 05:34
@d-mato d-mato marked this pull request as ready for review April 14, 2025 05:34
@d-mato d-mato mentioned this pull request Apr 14, 2025
3 tasks
@d-mato d-mato force-pushed the over-ssh-packages branch from 63d25c2 to 354c37b Compare April 17, 2025 01:42
@d-mato d-mato force-pushed the over-ssh-packages branch from 354c37b to 8667ff3 Compare April 17, 2025 01:47
@gjsjohnmurray
Copy link
Collaborator

@d-mato my PR #1471 may help resolve the CI failure in due course.

@Cremesis
Copy link

Hey, I was just stopping by to express my extreme interest in having this PR approved as soon as possible as it relates to a feature that is very important to me.
Thank you all for the work you are doing :)

Copy link
Collaborator

@gjsjohnmurray gjsjohnmurray left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am going to approve and merge this. If unresolved build/CI issues remain I will then attempt to resolve them.

@gjsjohnmurray gjsjohnmurray merged commit d397dc4 into mtxr:dev May 19, 2025
gjsjohnmurray added a commit that referenced this pull request Jun 2, 2025
* In @sqltools/base-driver the createSshTunnel method of AbstractDriver needs to be public rather than protected

* Use @sqltools/[email protected]

* Bump sqltools-driver-mssql version
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.

3 participants