[tor-commits] [tor/master] dirserv: Improve returned message when relay is rejected

[nickm@xxxxxxxxxxxxxx]

commit 211fe44e0735e33182fecb1582e4b8c9b59e256a
Author: David Goulet <dgoulet@xxxxxxxxxxxxxx>
Date:   Thu Feb 8 09:42:26 2018 -0500

    dirserv: Improve returned message when relay is rejected
    
    Explicitly inform the operator of the rejected relay to set a valid email
    address in the ContactInfo field and contact bad-relays@ mailing list.
    
    Fixes #25170
    
    Signed-off-by: David Goulet <dgoulet@xxxxxxxxxxxxxx>
---
 changes/ticket25170 |  5 +++++
 src/or/dirserv.c    | 10 ++++++++--
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/changes/ticket25170 b/changes/ticket25170
new file mode 100644
index 000000000..065213940
--- /dev/null
+++ b/changes/ticket25170
@@ -0,0 +1,5 @@
+  o Minor bugfix (directory authority, documentation):
+    - When a fingerprint or network address is marked as rejected, the
+      returned message by the authority now explicitly mention to set a valid
+      ContactInfo address and contact the bad-relays@ mailing list. Fixes bug
+      25170; bugfix on 0.2.9.1.
diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index fc6358a72..0f47a8398 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -417,7 +417,10 @@ dirserv_get_status_impl(const char *id_digest, const char *nickname,
 
   if (result & FP_REJECT) {
     if (msg)
-      *msg = "Fingerprint is marked rejected -- please contact us?";
+      *msg = "Fingerprint is marked rejected -- if you think this is a "
+             "mistake please set a valid email address in ContactInfo and "
+             "send an email to bad-relays@xxxxxxxxxxxxxxxxxxxx mentioning "
+             "your fingerprint(s)?";
     return FP_REJECT;
   } else if (result & FP_INVALID) {
     if (msg)
@@ -435,7 +438,10 @@ dirserv_get_status_impl(const char *id_digest, const char *nickname,
     log_fn(severity, LD_DIRSERV, "Rejecting '%s' because of address '%s'",
                nickname, fmt_addr32(addr));
     if (msg)
-      *msg = "Suspicious relay address range -- please contact us?";
+      *msg = "Suspicious relay address range -- if you think this is a "
+             "mistake please set a valid email address in ContactInfo and "
+             "send an email to bad-relays@xxxxxxxxxxxxxxxxxxxx mentioning "
+             "your address(es) and fingerprint(s)?";
     return FP_REJECT;
   }
   if (!authdir_policy_valid_address(addr, or_port)) {

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits