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

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