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

Convert documentation from rst to Markdown

It seems that many contributors already submit patches thinking that
rst is Markdown, which leads to incorrect and inconsistent formatting,
since the two are similar but not compatible.  By using the more well
known Markdown, we can perhaps hope to achieve better formatted
documentation contributions.

Also, since the web site uses Markdown as its source format, by using
Markdown here we can avoid the fragile conversion process altogether.

Remove outdated auth_file documentation

Once upon a time, one could point auth_file to pg_auth in the
PostgreSQL data directory, but this is long obsolete (gone as of
PostgreSQL 9.0), so remove mentions of it.

Prevent too long user name or password from client

In most cases, this wouldn't work anyway, because for example the user
wouldn't be found in pgbouncer, since userlist.txt doesn't permit too
long user names.  But in the case of PAM there was no such check, so
too long user names could be added by add_pam_user(), which would
truncate them, and then a subsequent search using the not-truncated
name wouldn't find it, causing duplicates to be added.

Reported-by: @achix
fixes #353

Merge pull request #356 from mnencia/issue-220

Avoid 'unexpected response from login query' after a postgres reload

Avoid 'unexpected response from login query' after a postgres reload

After a PostgreSQL reload, the backend could inform the frontend about
configuration changes by sending some ParameterStatus messages
together with the usual login query response. These messages can be
safely ignored in the login query response handler.

Closes: #220

Move comment to the right place

Put units on log message

age=N -> age=Ns

Merge pull request #329 from marcocitus/cancellations_fix

Do not wait server_login_retry for next connect if cancellation succeeds

Do not wait server_login_retry for next connect if cancellation succeeds

If postgres restarts while there are N cancellations in the queue,
pgbouncer is currently unavailable for at least N*server_login_retry
because it uses every new connection for one queued cancellation and
then waits server_login_retry before opening a new connection because
the last_connect_failed flag is still set to 1. This can lead to
prolonged downtime.

This changes fixes the issue by introducing a last_login_failed flag.
The last_connect_failed is now reset when a cancellation succeeds, such
that launch_new_connection no longer waits if pgbouncer manages to
connect, but has queued cancellations. The last_login_failed flag has
the same semantics as the last_connect_failed flag had previously, such
that check_fast_fail still rejects connections when there are no servers
available and the last login failed.

Add exit status to test scripts

When a test fails, exit with status 1, so automation can catch test
failures.  Also add a way to skip tests that cannot be run (mainly for
lack of sudo in this case).

Fix rare crash

SHOW FDS could in rare cases crash if a client had not been assigned
to a pool yet, causing a null pointer dereference.

fixes #311

doc: Update SHOW DATABASES documentation

Documentation of reserve_pool was missing.

fixes #322

Fix typos

fixes #322

Clarify file descriptor use log message

Make sure file descriptor info goes to the log

Delay the call to check_limits() until after daemonization so that the
message goes to the server log and is not printed to some other output
location.

Fix rst syntax

Improve shell script portability

Use printf instead of echo -n, for BSD compatibility, similar to
1b6db6c506cbed67fb4012a1217898a4b8e35252.

Fix psql invocation in tests

similar to 4f48ae3c7c7a3ae7aab4594420b0646ae27d4c7d, but for the ssl
tests

Fix up conntest test programs

Allow asynctest to run without an explicitly specified connection
string.  That way it can pick up the connection parameters from the
environment.  The run-conntest.sh driver script was already calling it
that way.

Fix the location of the pgbouncer binary in run-conntest.sh.

Improve test script robustness

Consistently exit with an error if the OS is not supported by the
firewall manipulation commands.  Previously, it would in some cases
print an error but not exit.

Fix typo in test script

Make log message more consistent

Travis CI: Run optscan.sh

optscan.sh: exit 1 on error

Update v1.9.0 release date

Fix typo

Bump libusual

v1.9

WAIT_CLOSE command

Fast close

Disconnect a server in session pool mode immediately if it is in
"close_needed" (reconnect) mode.  This ensures that the RECONNECT
command doesn't have to wait for session end.

RECONNECT command

Add close_needed column to SHOW SERVERS

This allows monitoring which server connections need reconnecting after
a configuration change.

Improve whitespace

Fix -Wimplicit-fallthrough warnings

in otherwise forgotten test program file

Minor whitespace fix in log message

test: Update MacOS firewall commands

Make commands in conntest.sh match the newer variants in test.sh.  Also
add a missing sudo.

doc: Clarify file descriptor formula

The previous phrasing made it look like max_pool_size etc. were
configuration settings.  Perhaps this way it's a bit clearer.

fixes #190

Make capitalization of log messages consistent

also minor typo and punctuation corrections

Merge pull request #255 from eulerto/reserve-pool-fix

Fix reserve_pool parameter

NEWS: Fix typos

fixes #296

doc: Clarify documentation of server_lifetime

fixes #305