Skip to content
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

Postgresql: .pgpass support #567

Open
hackenspp opened this issue Jan 24, 2024 · 3 comments · May be fixed by #666
Open

Postgresql: .pgpass support #567

hackenspp opened this issue Jan 24, 2024 · 3 comments · May be fixed by #666
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@hackenspp
Copy link

Connectorx does not support passwords stored in the .pgpass (or pgpass.conf in windows).
It's useful to avoid to specify it directly in the code or some sort of config file.

@aimtsou
Copy link
Contributor

aimtsou commented Apr 4, 2024

This can be achieved, I need to think how it can be given from the user that we use postgres in the conn variable.
After that we need some OS dependent code to see if pgpass exist and then to see if we read it how it gets passed. I believe it is the rewrite_conn method but I will need to dig deeper.

@surister
Copy link
Contributor

surister commented Jun 19, 2024

To some level we need to introduce some mechanism to do X depending on the database and the ability to identify the database (by uri.scheme I suppose)

This would also enable features like this to be easily implemented for other databases

@surister
Copy link
Contributor

@wangxiaoying Can you asign this to @aimtsou

@wangxiaoying wangxiaoying added enhancement New feature or request help wanted Extra attention is needed labels Jun 20, 2024
@surister surister linked a pull request Jul 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants