diff options
author | Andres Freund | 2015-07-28 19:39:32 +0000 |
---|---|---|
committer | Andres Freund | 2015-07-28 20:06:31 +0000 |
commit | 426746b93093a0fef53b23ce4b6421bed28e5c60 (patch) | |
tree | b4f53a6f5660b2856a7a177826cbb81efa85d75c /src/include | |
parent | 01f6bb4b2dcc571ad0cc6a404595de4b03157534 (diff) |
Remove ssl renegotiation support.
While postgres' use of SSL renegotiation is a good idea in theory, it
turned out to not work well in practice. The specification and openssl's
implementation of it have lead to several security issues. Postgres' use
of renegotiation also had its share of bugs.
Additionally OpenSSL has a bunch of bugs around renegotiation, reported
and open for years, that regularly lead to connections breaking with
obscure error messages. We tried increasingly complex workarounds to get
around these bugs, but we didn't find anything complete.
Since these connection breakages often lead to hard to debug problems,
e.g. spuriously failing base backups and significant latency spikes when
synchronous replication is used, we have decided to change the default
setting for ssl renegotiation to 0 (disabled) in the released
backbranches and remove it entirely in 9.5 and master.
Author: Andres Freund
Discussion: 20150624144148.GQ4797@alap3.anarazel.de
Backpatch: 9.5 and master, 9.0-9.4 get a different patch
Diffstat (limited to 'src/include')
-rw-r--r-- | src/include/libpq/libpq-be.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/include/libpq/libpq-be.h b/src/include/libpq/libpq-be.h index 6171ef3a1ff..caaa8b5f890 100644 --- a/src/include/libpq/libpq-be.h +++ b/src/include/libpq/libpq-be.h @@ -93,11 +93,6 @@ typedef struct #endif /* - * SSL renegotiations - */ -extern int ssl_renegotiation_limit; - -/* * This is used by the postmaster in its communication with frontends. It * contains all state information needed during this communication before the * backend is run. The Port structure is kept in malloc'd memory and is |