diff options
| author | Alexander Færøy <ahf@torproject.org> | 2020-06-30 13:55:39 +0000 |
|---|---|---|
| committer | Alexander Færøy <ahf@torproject.org> | 2020-06-30 13:55:39 +0000 |
| commit | 8444fbe904141d7954efe221b663efee1774df41 (patch) | |
| tree | 60469f39d42cc4b82e9ffec828038aaa99332526 /src | |
| parent | c3ad2a1d2367d6c0ba5c1c81491770056e881a77 (diff) | |
| parent | 8dc7ad1275a53cfe490128e7fb25b759dd7d88b5 (diff) | |
| download | tor-8444fbe904141d7954efe221b663efee1774df41.tar.gz tor-8444fbe904141d7954efe221b663efee1774df41.zip | |
Merge branch 'tor-github/pr/1793' into maint-0.3.5
Diffstat (limited to 'src')
| -rw-r--r-- | src/app/config/config.c | 13 | ||||
| -rw-r--r-- | src/test/test_options.c | 19 |
2 files changed, 21 insertions, 11 deletions
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."); diff --git a/src/test/test_options.c b/src/test/test_options.c index 66b0e7ef11..0e52967a23 100644 --- a/src/test/test_options.c +++ b/src/test/test_options.c @@ -485,7 +485,8 @@ test_options_validate__uname_for_server(void *ignored) #endif options_test_data_t *tdata = get_options_test_data( - "ORPort 127.0.0.1:5555"); + "ORPort 127.0.0.1:5555\n" + "ContactInfo nobody@example.com"); setup_capture_of_logs(LOG_WARN); MOCK(get_uname, fixed_get_uname); @@ -633,9 +634,11 @@ test_options_validate__contactinfo(void *ignored) ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg); tt_int_op(ret, OP_EQ, -1); expect_log_msg( - "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.\n"); + "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.\n"); tor_free(msg); free_options_test_data(tdata); @@ -645,9 +648,11 @@ test_options_validate__contactinfo(void *ignored) ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg); tt_int_op(ret, OP_EQ, -1); expect_no_log_msg( - "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.\n"); + "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.\n"); tor_free(msg); done: |