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

tokio-postgres: Set user to executing process' user by default. #1059

Merged
merged 3 commits into from
Aug 19, 2023

Conversation

ISibboI
Copy link
Contributor

@ISibboI ISibboI commented Aug 19, 2023

This mimics the behaviour of libpq and some other libraries (see #1024). This commit uses the whoami crate, and thus goes as far as defaulting the user to the executing process' user name on all operating systems.

Closes #1024

This mimics the behaviour of libpq and some other libraries (see sfackler#1024).
This commit uses the `whoami` crate, and thus goes as far as defaulting the user to the executing process' user name on all operating systems.
@ISibboI
Copy link
Contributor Author

ISibboI commented Aug 19, 2023

How should I go about the time package?

@sfackler
Copy link
Owner

You can bump the version we use in CI up to 1.67: https://github.com/sfackler/rust-postgres/blob/master/.github/workflows/ci.yml#L85

@sfackler sfackler merged commit f777f74 into sfackler:master Aug 19, 2023
4 checks passed
@sfackler
Copy link
Owner

Thanks!

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.

Make username default to current login name
2 participants