pgsql: Allow group access on PGDATA

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Allow group access on PGDATA
Date: 2018-04-07 21:46:20
Message-ID: E1f4vfA-0006cC-2r@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Allow group access on PGDATA

Allow the cluster to be optionally init'd with read access for the
group.

This means a relatively non-privileged user can perform a backup of the
cluster without requiring write privileges, which enhances security.

The mode of PGDATA is used to determine whether group permissions are
enabled for directory and file creates. This method was chosen as it's
simple and works well for the various utilities that write into PGDATA.

Changing the mode of PGDATA manually will not automatically change the
mode of all the files contained therein. If the user would like to
enable group access on an existing cluster then changing the mode of all
the existing files will be required. Note that pg_upgrade will
automatically change the mode of all migrated files if the new cluster
is init'd with the -g option.

Tests are included for the backend and all the utilities which operate
on the PG data directory to ensure that the correct mode is set based on
the data directory permissions.

Author: David Steele <david(at)pgmasters(dot)net>
Reviewed-By: Michael Paquier, with discussion amongst many others.
Discussion: https://postgr.es/m/ad346fe6-b23e-59f1-ecb7-0e08390ad629%40pgmasters.net

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c37b3d08ca6873f9d4eaf24c72a90a550970cbb8

Modified Files
--------------
doc/src/sgml/config.sgml | 17 ++++
doc/src/sgml/ref/initdb.sgml | 19 +++++
doc/src/sgml/ref/pg_basebackup.sgml | 6 ++
doc/src/sgml/ref/pg_receivewal.sgml | 6 ++
doc/src/sgml/ref/pg_recvlogical.sgml | 11 +++
doc/src/sgml/runtime.sgml | 26 +++++-
src/backend/bootstrap/bootstrap.c | 12 +--
src/backend/postmaster/postmaster.c | 87 ++++----------------
src/backend/tcop/postgres.c | 3 +-
src/backend/utils/init/globals.c | 9 +++
src/backend/utils/init/miscinit.c | 115 ++++++++++++++++++++++++---
src/backend/utils/misc/guc.c | 25 ++++++
src/bin/initdb/initdb.c | 29 ++++++-
src/bin/initdb/t/001_initdb.pl | 20 ++++-
src/bin/pg_basebackup/pg_basebackup.c | 26 ++++--
src/bin/pg_basebackup/pg_receivewal.c | 11 +++
src/bin/pg_basebackup/pg_recvlogical.c | 11 +++
src/bin/pg_basebackup/streamutil.c | 76 ++++++++++++++++++
src/bin/pg_basebackup/t/010_pg_basebackup.pl | 21 ++++-
src/bin/pg_ctl/pg_ctl.c | 12 ++-
src/bin/pg_ctl/t/001_start_stop.pl | 25 +++++-
src/bin/pg_resetwal/pg_resetwal.c | 10 +++
src/bin/pg_rewind/RewindTest.pm | 9 ++-
src/bin/pg_rewind/pg_rewind.c | 11 +++
src/bin/pg_rewind/t/002_databases.pl | 13 ++-
src/bin/pg_upgrade/pg_upgrade.c | 12 ++-
src/bin/pg_upgrade/test.sh | 16 ++--
src/common/file_perm.c | 72 ++++++++++++++++-
src/include/common/file_perm.h | 24 +++++-
src/include/miscadmin.h | 2 +
src/test/perl/PostgresNode.pm | 27 ++++++-
src/test/perl/TestLib.pm | 25 ++++++
32 files changed, 661 insertions(+), 127 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2018-04-07 21:57:30 Re: pgsql: Support partition pruning at execution time
Previous Message David Rowley 2018-04-07 21:40:39 Re: pgsql: Support partition pruning at execution time