Re: pgsql: Superuser can permit passwordless connections on postgres_fdw

Michael Paquier <[email protected]> Fri, 20 Dec 2019 21:02:08 +0900
Newsgroups gmane.comp.db.postgresql.devel.cvs
Message-ID <[email protected]>
Hi Andrew,

On Fri, Dec 20, 2019 at 05:55:10AM +0000, Andrew Dunstan wrote:
> Superuser can permit passwordless connections on postgres_fdw
> 
> Currently postgres_fdw doesn't permit a non-superuser to connect to a
> foreign server without specifying a password, or to use an
> authentication mechanism that doesn't use the password. This is to avoid
> using the settings and identity of the user running Postgres.
> 
> However, this doesn't make sense for all authentication methods. We
> therefore allow a superuser to set "password_required 'false'" for user
> mappings for the postgres_fdw. The superuser must ensure that the
> foreign server won't try to rely solely on the server identity (e.g.
> trust, peer, ident) or use an authentication mechanism that relies on the
> password settings (e.g. md5, scram-sha-256).
> 
> This feature is a prelude to better support for sslcert and sslkey
> settings in user mappings.

After this commit a couple of buildfarm animals are unhappy with the
regression tests of postgres_fdw:
 CREATE ROLE nosuper NOSUPERUSER;
+WARNING:  roles created by regression test cases should have names
 starting with "regress_"
 GRANT USAGE ON FOREIGN DATA WRAPPER postgres_fdw TO nosuper;
It is a project policy to only user roles prefixed by "regress_" in
regression tests.

These is also a second type of failure:
-HINT:  Valid options in this context are: [...] krbsrvname [...]
+HINT:  Valid options in this context are: [...]
The diff here is that krbsrvname is not part of the list of valid
options.  Anyway, as this list is build-dependent, I think that this
test needs some more design effort.
--
Michael
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEG72nH6vTowiyblFKnvQgOdbyQH0FAl38uEAACgkQnvQgOdby
QH2YYxAAiYGpBTBJ2PNEIgeq7Wd0foUN+K2XhVkSQbj45vkaWYDnitYrMw3tkw2v
+e/MwrF2TYvb2YD4ozvzw+dNQNn7pCS/SA2LhRrnOvAe/UAlQv6cp1fB9QVk9VUZ
zLmvVVF5jruf1TMO8UTVFcaJzdM88VOTm04Ae1mRMLLqhV6Dt5XRQTIHJPKk13nH
tP3HeoDSVsLP6LE0UnsoGGz4EXDdbLI/NEPlJ67D5yTD5ZRBb3XbOpudjsSrqGxX
D2lFTq0AXV1HKvyHWEfm32yKoaYOpLUKQ6kblmxKF1Z8jo9svpE7ibUPDxqRR8XV
JmFoLrQwPPNtFTN3zl5ueCxrax50ihghn+UxJ+7ablke0o3+T/If6BzmpAcHMJXm
jvIJA+VbrgqJeqOcKfXpnGmJR/YghZqABrOXkZMNY36+PSFhHwLTzrUfZc8RnBav
P5Zb6GtxXBy7xkYEqNJ8r4q62Mp+WcO9O4D4DIA+KffEKPlhIoWCmFI1Zxqs69su
sVHillfsPozlE6xXw2EDXiDMfzWP+49WJ2xpMBFU/5VuP885d6u0dU6n827EtEMo
/s+B5ffESG+m5IEU7AoiQ9x5swEoeIAEJWu/LcAwRuW92v3HJySTkr/oW61uYljb
esyB/eHkKNC4kE17QaeRF/vOcZ5Sq4c1S1RQALTO2KdgIQUK3pg=
=C8ju
-----END PGP SIGNATURE-----