diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-01-26 15:38:33 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-01-26 15:38:33 -0500 |
| commit | 9bbfca8fde6109ce84543b9d71bcbf1c3b144b29 (patch) | |
| tree | 4438d2ce3c58f3ebaa4c4ba8f075e3d772a71f5e /src | |
| parent | 7a47262ce6b3370564b005b05279b0e07fa15bba (diff) | |
Fix startup so that log prefix %h works for the log_connections message.
We entirely randomly chose to initialize port->remote_host just after
printing the log_connections message, when we could perfectly well do it
just before, allowing %h and %r to work for that message. Per gripe from
Artem Tomyuk.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/postmaster/postmaster.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c index 8ca048a7f60..38088360b80 100644 --- a/src/backend/postmaster/postmaster.c +++ b/src/backend/postmaster/postmaster.c @@ -4005,6 +4005,14 @@ BackendInitialize(Port *port) else snprintf(remote_ps_data, sizeof(remote_ps_data), "%s(%s)", remote_host, remote_port); + /* + * Save remote_host and remote_port in port structure (after this, they + * will appear in log_line_prefix data for log messages). + */ + port->remote_host = strdup(remote_host); + port->remote_port = strdup(remote_port); + + /* And now we can issue the Log_connections message, if wanted */ if (Log_connections) { if (remote_port[0]) @@ -4019,12 +4027,6 @@ BackendInitialize(Port *port) } /* - * save remote_host and remote_port in port structure - */ - port->remote_host = strdup(remote_host); - port->remote_port = strdup(remote_port); - - /* * If we did a reverse lookup to name, we might as well save the results * rather than possibly repeating the lookup during authentication. * |