Revert "Prevent running pg_basebackup as root"
authorMichael Paquier <michael@paquier.xyz>
Fri, 7 Feb 2020 01:51:17 +0000 (10:51 +0900)
committerMichael Paquier <michael@paquier.xyz>
Fri, 7 Feb 2020 01:51:17 +0000 (10:51 +0900)
This reverts commit 7bae0ad, as this is not ideal with the tar format,
and we may want to explore more options like what is done by tar with
some equivalents of --owner and --group, but for pg_basebackup.

Per complaints from Magnus Hagander and Stephen Frost.

Discussion: https://postgr.es/m/20200205172259.GW3195@tamriel.snowman.net

src/bin/pg_basebackup/pg_basebackup.c

index 4e12cdb44672d0d1facf173cd9b80433727633cb..2551cf38c91d8064d03c28598011ff8e760fe135 100644 (file)
@@ -2090,22 +2090,6 @@ main(int argc, char **argv)
                }
        }
 
-       /*
-        * Don't allow pg_basebackup to be run as root, to avoid creating files in
-        * the data directory with ownership rights incompatible with the
-        * postmaster.
-        */
-#ifndef WIN32
-       if (geteuid() == 0)                     /* 0 is root's uid */
-       {
-               pg_log_error("cannot be run as root");
-               fprintf(stderr,
-                               _("Please log in (using, e.g., \"su\") as the (unprivileged) user that will\n"
-                                 "own the server process.\n"));
-               exit(1);
-       }
-#endif
-
        atexit(cleanup_directories_atexit);
 
        while ((c = getopt_long(argc, argv, "CD:F:r:RS:T:X:l:nNzZ:d:c:h:p:U:s:wWkvP",