[COMMITTERS] pgsql: Disable prompting for passphrase while (re)loading SSL configfi - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Disable prompting for passphrase while (re)loading SSL configfi
Date
Msg-id E1cOSxs-0002qu-Kw@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Disable prompting for passphrase while (re)loading SSL config files.

OpenSSL's default behavior when loading a passphrase-protected key file
is to open /dev/tty and demand the password from there.  It was kinda
sorta okay to allow that to happen at server start, but really that was
never workable in standard daemon environments.  And it was a complete
fail on Windows, where the same thing would happen at every backend launch.
Yesterday's commit de41869b6 put the final nail in the coffin by causing
that to happen at every SIGHUP; even if you've still got a terminal acting
as the server's TTY, having the postmaster freeze until you enter the
passphrase again isn't acceptable.

Hence, override the default behavior with a callback that returns an empty
string, ensuring failure.  Change the documentation to say that you can't
have a passphrase-protected server key, period.

If we can think of a production-grade way of collecting a passphrase from
somewhere, we might do that once at server startup and use this callback
to feed it to OpenSSL, but it's far from clear that anyone cares enough
to invest that much work in the feature.  The lack of complaints about
the existing fractionally-baked behavior suggests nobody's using it anyway.

Discussion: https://postgr.es/m/29982.1483412575@sss.pgh.pa.us

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/1e942c7474d7b5e4bfa04918d2f68d95902f17b3

Modified Files
--------------
doc/src/sgml/runtime.sgml             |  9 ++++-----
src/backend/libpq/be-secure-openssl.c | 29 +++++++++++++++++++++++++++++
2 files changed, 33 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: [COMMITTERS] pgsql: Fix hot standby tests for sequence catalog change
Next
From: Bruce Momjian
Date:
Subject: [COMMITTERS] pgsql: Update copyright for 2017