[tor-commits] [tor/master] config: Warn if ContactInfo is not set

[nickm at torproject.org]

commit b755a489bdcdf56429c2c7e4fd313b2594dc68c0
Author: David Goulet <dgoulet at torproject.org>
Date:   Tue Feb 18 09:43:13 2020 -0500

    config: Warn if ContactInfo is not set
    
    Closes #33361
    
    Signed-off-by: David Goulet <dgoulet at torproject.org>
---
 changes/ticket33361     |  3 +++
 src/app/config/config.c | 13 +++++++++----
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/changes/ticket33361 b/changes/ticket33361
new file mode 100644
index 000000000..bc9715d6a
--- /dev/null
+++ b/changes/ticket33361
@@ -0,0 +1,3 @@
+  o Minor bugfix (relay, configuration):
+    - Now warn if the ContactInfo field is not set and mention that the relay
+      might get rejected if so. Fixes bug 33361; bugfix on 0.1.1.10-alpha.
diff --git a/src/app/config/config.c b/src/app/config/config.c
index cbca7d389..1d61b7631 100644
--- a/src/app/config/config.c
+++ b/src/app/config/config.c
@@ -3433,10 +3433,15 @@ options_validate(or_options_t *old_options, or_options_t *options,
     }
   }
 
-  if (server_mode(options) && !options->ContactInfo)
-    log_notice(LD_CONFIG, "Your ContactInfo config option is not set. "
-        "Please consider setting it, so we can contact you if your server is "
-        "misconfigured or something else goes wrong.");
+  if (server_mode(options) && !options->ContactInfo) {
+    log_warn(LD_CONFIG,
+             "Your ContactInfo config option is not set. Please strongly "
+             "consider setting it, so we can contact you if your relay is "
+             "misconfigured, end-of-life, or something else goes wrong. "
+             "It is also possible that your relay might get rejected from "
+             "the network due to a missing valid contact address.");
+  }
+
   const char *ContactInfo = options->ContactInfo;
   if (ContactInfo && !string_is_utf8(ContactInfo, strlen(ContactInfo)))
     REJECT("ContactInfo config option must be UTF-8.");