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
pgbouncer: use c-ares for name resolution #66336
Conversation
According to pgbouncer README.md file c-ares is the prefered way to resolve names for pgbouncer 1.10+: https://github.com/pgbouncer/pgbouncer/blob/master/README.md#dns-lookup-support The previous method (evdns) did not support EDNS0 and SOA lookups.
Anyone could see if this can be merged or has issues, apologizes for referencing you @danbst but you seem to be around postgres related PRs. |
interesting, does default APT pgbouncer package use c-ares? Can't find source for that |
@danbst I just checked on centos from the postgresql.org official repo and it does include c-ares:
On FreeBSD it is defined as an option: https://svnweb.freebsd.org/ports/head/databases/pgbouncer/Makefile?revision=507877&view=markup#l29 but then the version there is 1.9.0, but then according to the table I linked initially, the c-ares became the best DNS resolution method from version 1.10.0 onward, all other ones has some limitations. |
According to pgbouncer README.md file c-ares is the
prefered way to resolve names for pgbouncer 1.10+:
https://github.com/pgbouncer/pgbouncer/blob/master/README.md#dns-lookup-support
The previous method (evdns) did not support EDNS0 and SOA lookups.
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @ (no maintainers listed)