I found an error message added by de4d456b406bf502341ef526710d3f764b41e2c8.
When I incorrectly configured the primary_conninfo with the wrong
user, I received the following message on the server logs of both
servers involved in a physical replcation set.
[27022:walsender] FATAL: permission denied to start WAL sender
[27022:walsender] DETAIL: Only roles with the REPLICATION attribute may start
a WAL sender process.
I'm not sure if adding the user name in the log prefix is a common
practice, but without it, the log line might not have enough
information. Unlike other permission-related messages, this message is
not the something human operators receive in response to their
actions. It seems similar to connection authorization logs where the
user name is important. So, I'd like to propose the following
alternative.
[27022:walsender] DETAIL: The connection user "r1" requires the REPLICATION
attribute.
What do you think about this change?
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
diff --git a/src/backend/utils/init/postinit.c b/src/backend/utils/init/postinit.c
index 31d6a05426..31b33524ad 100644
--- a/src/backend/utils/init/postinit.c
+++ b/src/backend/utils/init/postinit.c
@@ -972,8 +972,8 @@ InitPostgres(const char *in_dbname, Oid dboid,
ereport(FATAL,
(errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),
errmsg("permission denied to start WAL sender"),
- errdetail("Only roles with the %s attribute may start a WAL sender process.",
- "REPLICATION")));
+ errdetail("The connection user \"%s\" requires the %s attribute.",
+ username, "REPLICATION")));
}
/*
