From d09dc8f6d4f53d20516678e45f0759a7b2ce1c8d Mon Sep 17 00:00:00 2001 From: favonia Date: Tue, 16 Jul 2024 19:35:28 -0500 Subject: [PATCH] docs: tweak warning about the root (#804) --- README.markdown | 3 ++- cmd/ddns/ddns.go | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/README.markdown b/README.markdown index 4a9efa9a..623cc306 100644 --- a/README.markdown +++ b/README.markdown @@ -104,7 +104,8 @@ services: user: "1000:1000" # Run the updater with a specific user ID and group ID (in that order). # You should change the two numbers based on your setup. - # This is optional but highly recommended; otherwise you run the updater as root! + # This is optional but highly recommended; otherwise, you will probably + # run the updater as a superuser (root), which is usually a bad idea. read_only: true # Make the container filesystem read-only (optional but recommended) cap_drop: [all] diff --git a/cmd/ddns/ddns.go b/cmd/ddns/ddns.go index 2d4e5e2a..1c8f2435 100644 --- a/cmd/ddns/ddns.go +++ b/cmd/ddns/ddns.go @@ -87,7 +87,7 @@ func realMain() int { //nolint:funlen // Check root privileges if syscall.Geteuid() == 0 { - ppfmt.Warningf(pp.EmojiUserWarning, "It is not recommended to run this tool as root") + ppfmt.Warningf(pp.EmojiUserWarning, "You are running this tool as root, which is usually a bad idea") } // Read the config and get the handler and the setter