Merge pull request #243 from gplv2/master

add clues for sysadmin concerning HBA issues

Merge pull request #229 from petere/crash-fix

Fix crash from memory overrun

Merge pull request #235 from petere/pgbouncer-ini-auth-type

pgbouncer.ini: Add missing auth_type's to comment

Add STATS_TOTALS/STATS_AVERAGES into HELP

And separate the STATS* commands to it's own line.

Missed by 656844e

Merge pull request #238 from 'tvondra/statistics-fixes'

Track separate stats for queries and transactions

add clues for sysadmin concerning HBA issues

Add SHOW STATS_TOTALS / STATS_AVERAGES as admin commands

We already had SHOW STATS, but after adding the wait_time colums it
got a bit too wide for regular screens. That's incovenient, as the
admin interface does not selecting only some of the columns.

But in most cases we're only interested either in totals or averages,
not both at the same time. So let's add commands showing only the
relevant subset of columns.

Track total time spent waiting for server

There already is maxwait column in pool details, showing how long
the oldest client is currently waiting. Which is useful, but it does
not allow us to easily break client time into the three basic
categories: busy, idle and waiting.

This adds tracking of total time spent waiting for a server, and
adds it as wait_time into SHOW STATS. The total value is simply
accumulated over time, the average value is 'per second' value.

The value is only updated when a client acquires a server, so if
a client is waiting indefinitely, the wait time won't be reflected
in the value. That would require inspecting the the list of waiting
clients, which seems like overkill. Moreover, that's what maxwait
is good for.

Track separate stats for queries and transactions

The documentation claimed avg_query tracks average query duration,
but it was only updated when switching into 'idle' state. So in
transaction or session pooling modes, it was actually the duration
of the whole transaction, including 'idle in transaction' time and
time spen executing (multiple) queries.

This was causing confusion, e.g. when the application was keeping
transactions open without executing any queries. In that case the
avg_query metric was increasing although the database was handling
queries just fine (without any slow-down).

Instead of just fixing the avg_query metric, it seems quite useful
to have timing data both for queries and transactions. So this
patch replaces

 - total_query_time
 - avg_query

with

 - total_xact_time
 - total_query_time
 - avg_xact_time
 - avg_query_time

This is also related to the definition of 'request' which was used
to compute the avg_query, and so it equal to a transaction. This
patch replaces the counter with two - one for transactions and one
for queries. As we also computed avg_req, the patch replaces

 - total_requests
 - avg_req

with

 - total_xact_count
 - total_query_time
 - avg_xact_count
 - avg_query_count

Note: This is likely to affect monitoring solutions, as the column
names in SHOW STATS change.

Corrects bits vs bytes

Change "b/s" to "B/s", because the units are bytes.

pgbouncer.ini: Add missing auth_type's to comment

Fix crash from memory overrun

Fixes: #228

Merge branch 'master' of https://github.com/pgbouncer/pgbouncer

Merge pull request #217 from mnencia/patch-1

dns_zone_check_period is available also with c-ares backend

Upgrade libusual

Dont set flags when doing with-cares=auto

Merge pull request #218 from mnencia/patch-2

Fix --with-cares=auto

Merge pull request #219 from rubenv/patch-1

Add missing space

Add missing space

Turns this:

    Stats: 21 req/s, in 579668 b/s, out 422757 b/s,query 84412 us

Into this:

    Stats: 21 req/s, in 579668 b/s, out 422757 b/s, query 84412 us

Fix --with-cares=auto

dns_zone_check_period is available also with c-ares backend

Merge pull request #196 from mhagander/auth_type_mode

Fix incorrect references to auth_mode

Merge pull request #202 from gbartolini/dev/142

Global management of 'auth_user'

Merge pull request #183 from jeduardo/master

Increased maximum length for postgres password

Merge branch 'master' of https://github.com/pgbouncer/pgbouncer

sbuf: check for EAGAIN in connect

seems it can be returned in win32

fixes: #188 #161

hba: call next_token after parsing filename (#209)

hba: parse_namefile did unnecessary free

Fixes: #209

Merge pull request #213 from procore/add_client_wait_time

Add per-socket wait times to SHOW SOCKET.

Merge pull request #212 from procore/add_maxwait_us

Add a maxwait microseconds column to SHOW POOLS.

Merge pull request #214 from secwall/master

Fix memleak on pg_hba file reload

Free user_name and db_name in hba rule_free to avoid memory leak on hba file reload

Add per-socket wait times to show commands.

Add a maxwait microseconds column to SHOW POOLS.

Fix a couple of typos

Signed-off-by: Gabriele Bartolini <gabriele.bartolini@2ndQuadrant.it>

Global management of `auth_user`

Fixes #142

Signed-off-by: Gabriele Bartolini <gabriele.bartolini@2ndQuadrant.it>

Fix incorrect references to auth_mode

The parameter is called auth_type, not auth_mode.

Improved comment on max password length

Added warning for max password length

Increased maximum length for postgres password

Merge pull request #178 from marco44/master

Add paused and disabled field to the SHOW DATABASES command

Add paused and disabled field on the show databases command

Merge pull request #157 from anayrat/master

Mention auth_user in pgbouncer.ini example

Merge pull request #171 from macobo/patch-1

Improve build instructions

doc: auth_query - mention that function is run in target db

Upgrade libusual

Mainly to get OpenSSL 1.1 fixes

Fixes: #159

Improve build instructions

Ran into this when building a docker image.

Merge branch 'master' of https://github.com/pgbouncer/pgbouncer

console: change server_encoding to UTF8 from SQL_ASCII

cannot see any benefit of keeping SQL_ASCII around here.

console: use UTF8 instead UNICODE as client_encoding

Seems UTF8 is proper encoding and UNICODE is alias.
Java JDBC driver gets confused by UNICODE.

Fixes: #165

Merge pull request #137 from waldyrious/patch-1

add license title

Merge pull request #162 from mhagander/typofix

Fix typo

Fix typo

Merge pull request #147 from cpatru/issue-141

Don't send ReadyForQuery on main_connection for CancelQuery requests

Mention auth_user in pgbouncer.ini example

Don't send ReadyForQuery on main_connection as it may cause clients to get out of sync.

Merge pull request #155 from mhagander/pam_thread_fix

Make pam_init happen in the child process when forking

Make pam_init happen in the child process when forking

Previously, pam_init would happen in the master process that then
quickly died, rendering pam authentication inoperative.

add license title

It's not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)

Add PAM authentication support

Adds support for PAM authentication. Since PAM API is synchronous, this
patch introduces PAM authentication thread which processes the
authentication requests in backgrouns so that the normal poll operations
of the main thread are not blocked.

This feature is turned off by default and has to be enabled by
--with-pam configure parameter.

Author: Ilya Staheev
Reviewed by Marko Kreen and Petr Jelinek

hba: Skip unparseable lines, keep parsing to the end.

Rejecting whole HBA file on failure seems bad usability.

Show warning but keep parsing.

Fixes #118

tls: reject client TLS request on unix socket

tls: do not ask TLS on unix socket

v1.7.2

deb: turn on full hardening, needs v9 dh

Sort tarball files too, just in case.

Disable cleanup.

Not useful for production loads, only for debugging.

Revert "Skip cleanup if there is takeover"

This reverts commit 7c03a6c8f8d9ffcc3f0152e74c629213487b6d64.

Skip cleanup if there is takeover

Show TLS backend version

Sync libusual

takeover: wait for pidfile to go away

Missing var declaration.

Sync libusual

Proper stale pidfile handling in check_pidfile.

- Don't use remove_pidfile as that also frees value.
- Check for error, exit if failed.

Merge remote-tracking branch 'main/master'

Always check for NULL cf_pidfile

It is used before config is loaded.

Merge pull request #112 from credativ/reproducible

Make build reproducible by dropping DBGVER handling

Make build reproducible by dropping DBGVER handling

Debian wants debug symbols for all builds (the binaries are stripped by
debhelper's dh_strip), but the "(compiled by...)" information added for
debug builds defeats the goal to make builds reproducible on the binary
level. Fix by dropping the DBGVER information which doesn't serve any
purpose if the build output is byte-identical on rebuild.

https://wiki.debian.org/ReproducibleBuilds

v1.7.1

doc: improve auth_user docs

doc: fix manpage section

todo cleanup

libusual: sync, change url

Sync libusual

Add TLS options to sample config file.

optscan: fix doc path

Rename sslmode "disabled" to "disable" as in libpq

sbuf: remove unused function

Sync libusual

Fix some format warnings under win64

tags: look deeper under libusual

doc: Improve server_reset_query description

Based on feedback in #110

Also remove obsolete suggestion for pre-8.3 postgres.

sbuf: in varify-ca/full checking should not be optional

Update libusual

cleanup: proparly free autodbs

sbuf: define handle_tls_handshake for non-tls build

Fixes: #101

Upgrade libusual

Clean allocated memory on exit.

Helps to track potential memory leaks.

Upgrade libevent