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

doc: Document that KILL requires RESUME afterwards

fixes #310

test.sh: Sort out netcat zoo

The previous code distinguished between Linux and non-Linux netcat.
This didn't stand the test of time, since Linux distributions now ship
multiple variants of netcat and other operating systems also provide
multiple options.  So instead try to detect the variant from the help
output.  Also, the previous uses of the -q and -w options behaved
strangely with some versions and/or were inconsistent with the
documentation.  So abandon those options and just rely on the existing
killall call.

The test_server_connect_timeout_establish test should now pass again on
hopefully most configurations.

Fix comments

Improve punctuation in stats message

Update code comment

doc: Document reconnect behavior on reload and DNS change

This is long-standing behavior, just make the documentation about it
more explicit.

doc: Add missing documentation for various SHOW fields

Various output fields from SHOW commands were not documented.

Fix -Wimplicit-fallthrough warnings

These warnings started showing up with GCC 7.  The code was already
correct, but the intent is now a bit clearer.  Compilation with recent
GCCs should now be warning-free again.

test: Restore userlist.txt after test run

test.sh: Wrap all sudo calls in CAN_SUDO check

One call had been missed.

.gitignore: Add test/pgdata/

Correct list of required software

In particular, pkg-config is required for normal builds, not just from
Git.

Normalize spelling of Git

doc: Document SET command

doc: Small improvement in quick start

Add an indication that the "SHOW HELP" example output is not complete.
Then we don't have to feel obliged to update it every time we add
something to the actual help output.

Merge pull request #301 from danielgustafsson/dg-null_deref

Fix a double-free and a NULL pointer deref

Avoid NULL pointer deref in parse_line

If we fail to calloc the rule buffer, going to failed: will deref
rule via the rule_free() call. Avoid by returning false directly
and bypass the failed: block.

Avoid double-free in parse_filename

If the fopen() call fails parse_namefile() frees the filename buffer
fn, and the caller does the same. Avoid freeing in parse_namefile()
and leave the cleanup to the caller.

Fix whitespace errors reported by Git

doc: Improve signals documentation

Add SIGUSR1 and SIGUSR2.  Fix grammar a bit.

Port mkauth.py to Python 3

fixes #263

Travis CI: Test make deb target

Fix mistake in test setup

This caused the test test_database_change to fail.

Fix psql invocation in tests

Always run psql with -X, otherwise some tests fail if there is something
in .psqlrc producing output.  Also fix some potential portability
problems in option syntax: Put non-option arguments last, don't mix
options with and without arguments.

v1.8.1

Travis CI: Add configure arguments matrix

Travis CI: Convert script to single YAML block

Otherwise the scripts are all run independently of each other, which
doesn't make sense.

Travis CI: Add test for building from dist tarball

Check that the distribution tarball can actually be built from, in case
it's missing a file.  Also check `make install` while we're here.

Add include/pam.h to distribution tarball

fixes #261

v1.8

doc: Fix rst syntax

Words ending in underscore are links.  This confuses pandoc (for the web
site) (but apparently not rst2man).  Escape with backslash to fix.

Display 0 ttl for stale DNS entries in SHOW DNS_HOSTS

Travis CI: Add dist target to script

Use strerror() instead of %m for portability