gcc braindamage: -Wformat-security is not turned on by -Wall -Wextra -Wformat

Increase size of various startup-related buffers.

Also bring them under single define.

Make listen(2) backlog argument configurable.

Dont let logging change errno.

With badly configured logging and higher logging levels, the code can
see wrong errno from syscalls.  So always retore original errno when logging.

Increase room for DateStyle storage.

If value is larger than buffer, it stays empty and libpq will drop connection.

Explicitly detect V2 protocol request and close it with proper msg.

Move fast-fail relaunch logic around.

Do it together with fast-fail disconnect.  When kept separately
it can launch even if there are no new clients coming.
This causes problem with permanently dead dbs/users, as the
retry will not stop if all clients are gone.

v1.3.2

Log pooler-sent errors as warnings not plain info, as they signify problems.

log message: statement timeout -> query timeout, to match config var

Don't let clients queue in case no servers and connections fail.

That allows to avoid unnecessary delays in hopeless situations.

Introduce a new configuration setting: query_wait_timeout

If a query spends more than query_wait_timeout seconds waiting
it is disconnected. This prevents unresponsive servers from
wasting resources.

Patch by Martin Pihlak

doc/confix.txt: uncoonfuse newer asciidoc

Check error from event_init() to eliminate warning on strict build

Add new patches to NEWS

sbuf: track exact wait state to avoid double event_del()

In case of invalid state, show value in error msg

test/test.ini: turn off query_timeout by default

prepare for 1.3.2rc1

todo update

Apply client_login_timeout to clients waiting for welcome pkt

Otherwise the clients stay waiting infinitely, unless there is query_timeout set.

This only applies for servers that have never been up.  If server has
been connected once, the query_timeout is only thing that kills clients,
as clients are let freely log in after first connect to server.

win32: allow user to override service account

The automatic decision between Local System and Local Service
was not reliable enough.  And anyway, it's better to let user
pick proper account.

Clarify few login-related log messages

Disable maintenance during SUSPEND

Otherwise some timeout may activate and close connection
that is already reported to other side;

console: fix string format in datarow packet

pgbouncer used to store strings with final '\0', which should not be there.

It was not noticed thus far as any C clients did not saw anything wrong.

Takeover code depends on them being zero-terminated, use a hack to make them so.

doc/config.txt improvements

Patch by Greg Sabino Mullane, applied with minor modifications.

doc/usage.txt improvements

Patch by Greg Sabino Mullane, applied with minor modifications.

v1.3.1

docs: mention console behavious when auth_mode=any

When auth_mode=any, allow anybody in as admin.

Because auth_mode=any does not store original username,
the usual username based checks (admin_users/stats_users)
do not work.

todo: clarify

faq/ssl: pgsql protocol patch was applied to stunnel 4.27

v1.3.1rc1

todo update

sbuf.c: Fix connection hang if sbuf_loopcnt is set.

If query or result length nearby of multiple of (pktlen*sbuf_loopcnt)
[10k by default], it could stay waiting for more data which will not
appear.

Fix it by resyncing and final processing before exiting loop.

usage.txt: Time units are microseconds, not milliseconds

admin.c: mention SHOW STATS in help msg

Fix potential use of old connections after config change

Despite detecting and tagging them for close on config change,
the closing happened only in long-term maintenance.  So there
was time where they could still be reused by clients.

Fix this by strict check in find_server()

Fixed adjustment of a WSAStartup position problem.
It happen the hostname problem in the service start.
Per report Sachin Srivastava. and, thanks suggestions of Marko.

Fixed the Service parameter path name problem.
Per report from Sachin Srivastava.

Fix broken alignment in src/slab.c

Alignment macro was buggy, thus resulting in bad alignment.
This means speed hit on x86, but may result in crashes
on arches which are strict on alignment, unless the macro
is pre-defined in OS headers.

No problems from the field have been reported, thus maybe
even the bad alignment happened to be aligned?

SHOW DATABASES had wrong column count.

The new res_pool field was not mentioned in data rows.

update docs and default config

release 1.3

win32: improve service description

Hiroshi Saito

Apply ISC license clarification, bump copyright years.

Clarification:

  for f in */*.[ch]; do echo -e "%s;and distribute;and/or distribute;\nwq" | ed "$f"; done

Details at: http://en.wikipedia.org/wiki/ISC_licence

duh, seems we did not include COPYRIGHT file in tgz

README: mention reserve_pool feature

add missing win32 files to tgz

event_loopbreak compat for libevent 1.3

tag 1.3rc1

README: win32 info

doc/Makefile: build also html for README

NEWS: expand autodb and win32 items

win32: make service_name alias for job_name

win32 todo

NEWS: mention event_del problem

add few overlooked contributors

sbuf_close: dont retry event_del() if it fails, just proceed with cleanup.

The retry could have been bug in pgbouncer - although then the
question is that how to survive ENOMEM from event handlers that
may need allocation on event_del().

But this does not explain why it fails first time.  Seems there
is something funny going on in libevent.

win32: simpler service handling, cleanups

- one pgbouncer instance per service, in-process
- read service name from config file

cleanups:

- use FormatMessage for strerror()
- error out on wide-char build
- tag stdout/err _IONBF, they seem fully buffered by default

some more cleanups:

- avoid NULL values for config variables
- cf_shutdown=2 means fast shutdown
- export cf_config_file

minor cleanups

- avoid use of non-const fmt string
- avoid duplicate def of sbuf_len

draft news for 1.3

Reserve pool implementation.

Allow to increased pool size in case main pool size
cannot serve clients anymore.

This allows to continue use of small pool_size, which
smooths small peaks, while also handling large peaks.

(Eg, one bouncer server down, others need to handle load.)

check if connection was accepted before logging

win32: make asynctest compile again

By Hiroshi Saito

win32: add -A to def generation

win32: no need for -subservice switch, instead use -q

dlltool seems to need --export-all-symbols to make .def file

pgbevent: make g_module static var

win32: more fixes

- proper message id
- fix bugs in cmdline validation

locatime weirdness wasnt windows specific...

win32: tm_mon fix for localtime() (Hiroshi Saito)

win32: rename support files

win32: more cleanups

- emulate fcntl instead using #ifdef
- getsockopt() and select() also need wrapping
- move syslog declarations into compat_win32.h

win32: implement kill()

Patch by Hiroshi Saito, applied with some modifications.

win32: implement sendmsg, move cmdline handling away from main.c

win32: fix locatime

win32: errno fixes

- switch errors around to correspond to what gets returned
- somewhat working strerror()

win32: more fixes

- move socket errors back to errno
- don't call closesocket() on plain files

win32: change name of eventmsg

remove the mc.exe detection, it not available anyway

Quick win32 fixes.

It not only compiles now, but even boots..

remove unnecessary CVS ids, update AUTHORS

win32 support (wip)

Original patch by Hiroshi Saito, applied with some cleanup

Make crypt authentication optional.

It was removed from PostgreSQL 8.4 sources.

Dynamic database creation for non-defined databases.

Activated by "*" entry in [databases] section.

Patch by David Galoyan

asynctest improvements

- add $(LIBS) to link line
- allow custom query
- define fatal_noexit() for CASSERT builds

add milliseconds to log timestamps

New tunable 'sbuf_loopcnt' to limit time spent on one socket.

In some situations - eg SMP server, local Postgres and fast network -
pgbouncer can run recv()->send() loop many times without blocking
on either side.  But that means other connections will stall for
a long time.  To make processing more fair, limit the times
of doing recv()->send() one socket.  If count reaches limit,
just proceed processing other sockets.  The processing for
that socket will resume on next event loop.

Thanks to Alexander Schöcke for report and testing.

debug: log connection endpoints

Change default port number from 6000 to 6432.

6000 is conflicts with X, 6432 seems free and is already used by Debian.

quiet compiler warning about ptr->int conversion

stop playing with -fomit-frame-pointer

Default to --enable-debug

To get the old behaviour - stripped binary and -fomit-frame-pointer
in CFLAGS, --disable-debug must be used.

Allowing debugging by default seems saner.

Put own includes at the startt of CPPFLAGS

Otherwise with --with-libevent=/usr wrong includes may be used.

Per report from Bjorn Munch

compact md5 implementation

full quoting for database names

tag head as 1.3

include userlist.txt in tgz