Logging: do not log a missing proxy address, on delivery non-proxied.
authorJeremy Harris <jgh146exb@wizmail.org>
Wed, 26 Sep 2018 09:59:05 +0000 (10:59 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Wed, 26 Sep 2018 09:59:05 +0000 (10:59 +0100)
Broken-by: e6d2a9894d
doc/doc-txt/ChangeLog
src/src/globals.c

index 5a04b1bdc915569b386b15c483229b1f87de86d4..8d64dd4b0efa1ba67c46a016130eccdb53a2d361 100644 (file)
@@ -125,6 +125,10 @@ JH/26 Bug 2311: Fix DANE-TA verification under GnuTLS.  Previously it was also
       TLSA as an anchor.  Checking the name on the leaf cert against the name
       on the A-record for the host is still done for TA (but not for EE mode).
 
+JH/27 Fix logging of proxy address.  Previously, a pointless "PRX=[]:0" would be
+      included in delivery lines for non-proxied connections, when compiled with
+      SUPPORT_SOCKS and running with proxy logging enabled.
+
 
 Exim version 4.91
 -----------------
index 340c45187c72e4b9e0296887f420f27243e449f8..c18f6c163f4c9b048b21094fc8382a7fe840e054 100644 (file)
@@ -1184,10 +1184,10 @@ int     process_info_len       = 0;
 uschar *process_log_path       = NULL;
 
 #if defined(SUPPORT_PROXY) || defined(SUPPORT_SOCKS)
-uschar *hosts_proxy            = US"";
-uschar *proxy_external_address = US"";
+uschar *hosts_proxy            = NULL;
+uschar *proxy_external_address = NULL;
 int     proxy_external_port    = 0;
-uschar *proxy_local_address    = US"";
+uschar *proxy_local_address    = NULL;
 int     proxy_local_port       = 0;
 #endif