diff options
| author | Andrew Dunstan <andrew@dunslane.net> | 2020-01-13 18:08:09 +1030 |
|---|---|---|
| committer | Andrew Dunstan <andrew@dunslane.net> | 2020-01-13 18:08:09 +1030 |
| commit | cebf9d6e6ee13cbf9f1a91ec633cf96780ffc985 (patch) | |
| tree | 1726435c36284b74889b59ca3bb8c50abc669e6d /contrib/postgres_fdw/sql/postgres_fdw.sql | |
| parent | 4e514c6180fbf71cf7a0171867c828c63afd1c37 (diff) | |
| download | postgresql-cebf9d6e6ee13cbf9f1a91ec633cf96780ffc985.tar.gz postgresql-cebf9d6e6ee13cbf9f1a91ec633cf96780ffc985.zip | |
Only superuser can set sslcert/sslkey in postgres_fdw user mappings
Othrwise there is a security risk.
Discussion: https://postgr.es/m/20200109103014.GA4192@msg.df7cb.de
Diffstat (limited to 'contrib/postgres_fdw/sql/postgres_fdw.sql')
| -rw-r--r-- | contrib/postgres_fdw/sql/postgres_fdw.sql | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/contrib/postgres_fdw/sql/postgres_fdw.sql b/contrib/postgres_fdw/sql/postgres_fdw.sql index f2a4089e9d3..acd7275c729 100644 --- a/contrib/postgres_fdw/sql/postgres_fdw.sql +++ b/contrib/postgres_fdw/sql/postgres_fdw.sql @@ -2567,6 +2567,7 @@ SELECT * FROM ft1_nopw LIMIT 1; -- Unpriv user cannot make the mapping passwordless ALTER USER MAPPING FOR CURRENT_USER SERVER loopback_nopw OPTIONS (ADD password_required 'false'); + SELECT * FROM ft1_nopw LIMIT 1; RESET ROLE; @@ -2579,6 +2580,12 @@ SET ROLE regress_nosuper; -- Should finally work now SELECT * FROM ft1_nopw LIMIT 1; +-- unpriv user also cannot set sslcert / sslkey on the user mapping +-- first set password_required so we see the right error messages +ALTER USER MAPPING FOR CURRENT_USER SERVER loopback_nopw OPTIONS (SET password_required 'true'); +ALTER USER MAPPING FOR CURRENT_USER SERVER loopback_nopw OPTIONS (ADD sslcert 'foo.crt'); +ALTER USER MAPPING FOR CURRENT_USER SERVER loopback_nopw OPTIONS (ADD sslkey 'foo.key'); + -- We're done with the role named after a specific user and need to check the -- changes to the public mapping. DROP USER MAPPING FOR CURRENT_USER SERVER loopback_nopw; |