-
Notifications
You must be signed in to change notification settings - Fork 350
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
feat: add support for unix sockets (linux, mac, and windows) #1182
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this is failing on Windows - what do we need to do different for windows support?
This reverts commit 1eb72ce.
I've updated the Postgres e2e test to use pgx to help ease the transition to v2. As part of that, I discovered pgx does not support Unix sockets on Windows, but fixing that would be only a small amount of work. I've filed jackc/pgx#1199 and otherwise disabled the test on Windows for now. |
Added supported for Unix sockets for Windows in pgx here: jackc/pgconn#114 and there's already a new release with the patch in it. I've updated the PR here to use that latest version. |
This is an adaptation from GoogleCloudPlatform/cloud-sql-proxy#1182
This is an adaptation from GoogleCloudPlatform/cloud-sql-proxy#1182
Fixes #84