summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Goulet <dgoulet@torproject.org>2020-02-18 09:43:13 -0500
committerNick Mathewson <nickm@torproject.org>2020-03-12 12:43:00 -0400
commitb755a489bdcdf56429c2c7e4fd313b2594dc68c0 (patch)
tree21f7ad8d5251f967141e295c3d11a85cb7abff7d
parentb9c7c61ea5233854ff83257a8bc530b7e0a50351 (diff)
downloadtor-b755a489bdcdf56429c2c7e4fd313b2594dc68c0.tar.gz
tor-b755a489bdcdf56429c2c7e4fd313b2594dc68c0.zip
config: Warn if ContactInfo is not set
Closes #33361 Signed-off-by: David Goulet <dgoulet@torproject.org>
-rw-r--r--changes/ticket333613
-rw-r--r--src/app/config/config.c13
2 files changed, 12 insertions, 4 deletions
diff --git a/changes/ticket33361 b/changes/ticket33361
new file mode 100644
index 0000000000..bc9715d6a1
--- /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 cbca7d3899..1d61b76310 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.");