v1.12.0

Bump libusual

Fix some scan-build warnings

Update website URL

Documentation proof reading and copy editing

Fix test_server_login_retry stability

The problem is that server_login_retry is only applied when the
backend server is down, but not when it returns an error upon
connection (e.g., authentication error).  In that case, the client
sees the error immediately.  Be that as it may, it breaks this test
sometimes, if a retry attempt hits the window where the server reports
"database is starting up".  (This window should be tiny since no
notable recovery should be happening here, but it happens.)  To fix,
adjust the settings so that the retries happen well away from the
backend server startup phase.

Accept SCRAM channel binding enabled clients

Add support to the SCRAM exchange for clients that support channel
binding, such as PostgreSQL version 11 and beyond.  If such a client
encounters a PgBouncer server that does not support channel binding,
it will send a channel binding flag 'y', meaning the client supports
channel binding but thinks the server does not.  But PgBouncer
erroneously did not accept that flag.  This would cause connections to
fail if a PostgreSQL version 11 client connects to a PgBouncer with
SCRAM authentication over SSL.

adapted from PostgreSQL 218b024a7ec866ec62abb5c2fb4eb9108bb5fc0f

Reported-by: Andrew Dunstan

Improve fast-fail code comments and debug logging

no changes in functionality

Add resolv_conf option to set custom resolv.conf file

closes #379

dnslookup: Consistently use log_error during impl_init()

This was done inconsistently between the different backends.  Some
used log_warning() for conditions that would ultimately be fatal.

Fix inappropriate uses of fatal_perror()

Use just fatal() instead when errno is not being set.

Fix terminology

PAM is authentication, not authorization.

Remove slightly outdated comment

We don't need to keep the list of DNS backends here, we already have
that documented elsewhere.

Clean up configure help strings

Make the style consistent with the built-in help strings.

Fix small markup issue

<foo> is a URL in Markdown, so this was misformatted.

Give more detailed error messages in tune_socket()

Give details on the actual system call that failed, rather than one
generic message for all.  Style similar to add_listen().

These "tune_socket() failed" messages occasionally appear in user
reports, but so far it was difficult to narrow down the issues.

Add so_reuseport option

Adds an option to specify whether to set the socket option
`SO_REUSEPORT` on TCP listening sockets.  On some operating systems,
this allows running multiple PgBouncer instances on the same host
listening on the same port and having the kernel distribute the
connections automatically.  This option is a way to get PgBouncer to
use more CPU cores.

closes #345

Reorder suspend_timeout in documentation

It was grouped differently in the documentation and in the sample
config file.

Tweak version string

Use Autoconf's PACKAGE_STRING for the --version and SHOW VERSION
output instead of our own macro.  This changes the output format
slightly, but we might as well change to the standard format while
we're changing around SHOW VERSION.

Send SHOW VERSION as row instead of NOTICE

This makes it easier to consume and is consistent with other SHOW
commands.

patch by @eulerto, @davidfetter; documentation and tests by @petere

fixes #254, fixes #258, fixes #321

Fix test exit status

The script was not reporting a total exit status of failure when a
test function would return a non-zero code other than 1.

Travis CI: Remove redundant configuration

Remove --with-openssl variant, since it's the default.

Fix reporting of adns implementation in configure

This was slightly broken by 748c83e5d8b05448725ce8a69213ed620a53d431
in the sense that configure would report both, say, c-ares and evdns
as "yes" without indication which one would actually be used.  Clean
that logic up a bit and report the actually used implementation at the
end of configure in a clearer way.  Also improve the documentation in
README.md around this.

Also clean up the build matrix in .travis.yml and remove a redundant
configuration.

Automatically generate debian/changelog

This avoids another manual step during tarball wrapping.

AppVeyor: Build on Windows with -Werror

Bump libusual

Travis CI: Use correct apt repository

Change from xenial-pgdg to bionic-pgdg.

Fix some compiler warnings on Windows 64-bit

For event_set() callbacks, use evutil_socket_t instead of int as the
first argument.  This is required for 64-bit Windows; on all other
platforms, the two types are the same.

This requires libevent version 2.

Note that the rest of the socket code still uses int everywhere, so
Windows 64-bit is still probably not going to work correctly.

Remove use of libusual event module

We use libevent directly now and use pkg-config to look it up.

The libusual event module was from a time when libevent was new and
evolving, but now we don't need it anymore and can use libevent
directly.

Remove dubious const qualifiers

These depend on libusual's redefinition of free(), which we should get
rid of.

see #371

Remove support for libevent version 1.x

This is long obsolete and stands in the way of some future
improvements.

Remove broken assertions

These haven't worked since 035b15e3707bb295142830a27ac36f6997cb3861.

Fix issue with PAM users losing their password

The normal flow when starting a client connection is that set_pool()
is called with an empty password ("").  For normal users, the password
was already set earlier from auth_file or auth_query, so this empty
password is ignored.  For PAM users, the empty password is stored but
the real password is set later when authentication is complete.

The problem in the PAM case is that for the next client connection,
this would overwrite the stored password with an empty password until
the real password would then be re-added later.  If the client
authentication doesn't complete for whatever reason (perhaps server is
down and fast-fail is active), then the correct password is never set.
This would then have clobbered the user's password that might be
useful for server authentication.  There are probably other failure
scenarios.

To fix, call set_pool() with a NULL password instead, and teach
add_pam_user() not to overwrite an existing password if the argument
is NULL.

reported, analysis, and fix proposal by @achix

fixes #285

Remove no longer used macro

Send stats columns as numeric instead of bigint

Internally, most statistiscs are kept as uint64_t.  Sending those
with a row descriptor that claims they are bigint can lead to problems
if uint64_t values can overflow the signed 64-bit integer range.  Then
a client library that wants to convert the value to its locally
appropriate signed 64-bit integer type would run into errors.

based on patch by @stanhu

fixes #360 #401

Remove references to PostgreSQL wiki

The information is now all on the PgBouncer web site.

Add make check target

This is a standard way to run all tests.

Travis CI: Update to bionic instead of xenial

Enable OpenSSL for Valgrind run, since it should work now (see
66b6dc2c31f7360e11347b20216c6c7ec5ce13cf).

Disable ElectricFence, seems broken:
<https://answers.launchpad.net/ubuntu/+question/674132>

Enable -Werror for c-ares build.  Warning about broken IPv6 support
should be gone now.

Travis CI: Shell fix

Newer cd versions apparently complain about too many arguments, which
was a problem here, since pgbouncer-* expands to both the directory
name and the tarball name.

dnslookup: Improve warning message phrasing

dnslookup: Fix compiler warnings from unused functions

v1.11.0

Add simple test for all SHOW commands

Add SASL and SCRAM support

Add support for SASL/SCRAM-SHA-256 authentication for clients and servers.

Import a few files from PostgreSQL core (commit REL_11_5) to support
SCRAM.

resolves #208

Recognize GSSENCRequest packet

This is a new startup packet type introduced in PostgreSQL 12.  In
PgBouncer, we'll for now reject GSS encryption attempts.

Without this, PgBouncer would reject such connections with "bad packet
header" errors, requiring the client to reconnect before being able to
do the normal startup.

Bump libusual

test: Restart pgbouncer for every test

There are too many cases where existing pool states lingers between
one test and another, making some test results unreliable.  It's safer
to just restart the whole process to get a clean start.

This also lets us mark the pgbouncer log file with where a test began
and ended.

Test against multiple PostgreSQL versions

This allows testing versions with older and newer authentication
behavior.

Disable PAM for Valgrind build

It creates mysterious errors that I don't want to deal with right now.

More useful Valgrind options

Make sure leaks actually result in an error report.

Remove --read-var-info=yes since it causes problems on some systems.

Add tests for taking password from auth_file

Add test for md5 authentication with plain-text password stored

Now we have tests for all four combinations of plain-text and md5
passwords and plain and md5 authentication.

Clean up example and test configuration files

Consistently comment out settings that are the default, update various
documentation comments, make commenting consistent.

In configuration files for tests, remove most comments, so we don't
have to keep syncing them with the main example file.

doc: Update link to pg_hba.conf documentation

Also rephrase the text a bit so that it doesn't need to be updated for
every authentication method change in PostgreSQL or PgBouncer.

README: Wording improvements

Add AppVeyor configuration for Windows

Travis CI: Add test run with Valgrind

Note: Valgrind on Ubuntu xenial is broken for OpenSSL[0], so don't
configure that right now.  Travis CI on bionic still has issues[1], so
we're stuck here for now.

[0]: https://bugs.launchpad.net/ubuntu/+source/valgrind/+bug/1574437
[1]: https://travis-ci.community/t/apt-addon-doesnt-run-in-bionic/4061

Update sample pgbouncer.ini for log_stats addition

Add option to disable printing stats to log

Add option log_stats that can be used to disable printing stats to
log.  Stats are still updated every stats_period, for SHOW commands.

Author: @weastur

fixes #287

Handle auth_type=password when the stored password is md5

This matches what a PostgreSQL server would do.

Author: @hashbrowncipher

fixes #129

Add tests for plain-text password authentication

Add [users] section to example config file

Also add some missing documentation about the [users] section.

Reported-by: @brunomgalmeida
fixes #330

Fix typo

Substitute version into man pages automatically

test: Add -w option to pg_ctl invocation

This is the default in recent PostgreSQL versions.  This change just
makes the script behavior consistent across versions.  We can now also
remove some sleep calls to make the tests start faster.

Reorganize AUTH_* symbols

Separate numerically into protocol codes and internal codes.  This
makes room for protocol codes currently not supported.

Travis CI: Add test run with ElectricFence

Tweak test timing to make it more stable

configure.ac: Update AC_INIT invocation

Add bug report and URL information, update PgBouncer capitalization.

Improve debug message

Fix debug messag

It was using an uninitialized variable.

Travis CI: Also run ssl tests

Sync ssl/test.sh with test.sh

Make locations for logs and pid files consistent.

Remove ssl test data from git

The test script deletes it anyway before use, so it's of no use here.

Add tests for md5 authentication

Fix shell syntax in test suite

Using return outside a function doesn't make sense.

Add some debugging for failed tests

Print the log file of the failed test if the test fails.  This helps
analyzing failures when run in a CI environment.

Travis CI: Run tests as part of build

This required a fair amount of tweaking to get the tests stable
enough, but it seems good now.

Tweak some test timing to make the tests more stable

Rewrite test_server_connect_timeout_establish using pre_auth_delay

This avoids having to deal with strange behaviors and portability
issues from netcat.

Since ALTER SYSTEM is being used, this now requires at least
PostgreSQL 9.4 for the test.

Fix wait time computation with auth_user

When using auth_user, the transition to the CL_WAITING_LOGIN state
would not initialize the client->wait_start field.  This would either
lead to garbage values being recorded, or under assertions enabled it
would crash in activate_client().

(test_auth_user was actually reproducing this problem, but a crash
requires assertions enabled and new memory being all zero, so it was
difficult to catch it.)

Author: @pinaraf

see #393

v1.10.0

Bump libusual

Rewrite man page filter to work independent of Pandoc

This allows it to work with really old Pandoc versions that don't have
the --filter option (e.g., on CentOS 6).  This just makes it a plain
text-munging filter script.

Make stress.py compatible with Python 3

Do not depend on Pandoc 2.0

Replace the Lua filter with one written in Python, so it can work with
older Pandoc versions.  Import pandocfilters.py package from Pandoc to
help with that.

Improve pid file processing

Remove some useless code that triggers a TOCTOU warning from
Coverity.  Make error messages more accurate.

Add support for TLSv1.3 enabling and disabling

It already worked automatically; this just adds the options to enable
and disable the protocol version explicitly.

Add missing options TOTALS and USERS

These were not included in the output of SHOW HELP.

Author: @davidfetter

fixes #388

Fix usage typo

a online restart -> an online restart

Improve error message

Fix idle_transaction_timeout calculation

idle_transaction_timeout should count from the last request of the
server, because the server sent the idle information.  The code
previously used the last request of the client, which could lead to
premature timeouts.

fixes #125

Make error messages more precise

Tweak some debug messages for clarity

doc: Document remaining SHOW commands

Some of these present internal information that we don't want to
document fully, but at least mentioning them for completeness seems
reasonable.

fixes #394

doc: Various cosmetic improvements

Add documentation for available test suites

fixes #83

pycodestyle (PEP8) pass over Python scripts

stress.py: Convert to psycopg2

It was still on the ancient psycopg module.

stress.py: Update port number in test script

from 6000 to 6432