[tor-commits] [tor] branch main updated: vote AuthDirMaxServersPerAddr in consensus params

[gitolite role]

This is an automated email from the git hooks/post-receive script.

dgoulet pushed a commit to branch main
in repository tor.

The following commit(s) were added to refs/heads/main by this push:
     new 36612b9bf8 vote AuthDirMaxServersPerAddr in consensus params
     new 73eab76e6d Merge branch 'maint-0.4.7'
36612b9bf8 is described below

commit 36612b9bf8f432ccfc5fc637dd86463b84ab4882
Author: Roger Dingledine <arma at torproject.org>
AuthorDate: Sun Feb 12 15:50:55 2023 -0500

    vote AuthDirMaxServersPerAddr in consensus params
    
    Directory authorities now include their AuthDirMaxServersPerAddr
    config option in the consensus parameter section of their vote. Now
    external tools can better predict how they will behave.
    
    In particular, the value should make its way to the
    https://consensus-health.torproject.org/#consensusparams page.
    
    Once enough dir auths vote this param, they should also compute a
    consensus value for it in the consensus document. Nothing uses this
    consensus value yet, but we could imagine having dir auths consult it
    in the future.
    
    Implements ticket 40753.
---
 changes/ticket40753           | 5 +++++
 src/feature/dirauth/dirvote.c | 8 ++++++++
 2 files changed, 13 insertions(+)

diff --git a/changes/ticket40753 b/changes/ticket40753
new file mode 100644
index 0000000000..c5dc76b006
--- /dev/null
+++ b/changes/ticket40753
@@ -0,0 +1,5 @@
+  o Minor features (directory authorities):
+    - Directory authorities now include their AuthDirMaxServersPerAddr
+      config option in the consensus parameter section of their vote. Now
+      external tools can better predict how they will behave. Implements
+      ticket 40753.
diff --git a/src/feature/dirauth/dirvote.c b/src/feature/dirauth/dirvote.c
index 1bb4fd7de1..0591125d51 100644
--- a/src/feature/dirauth/dirvote.c
+++ b/src/feature/dirauth/dirvote.c
@@ -4878,6 +4878,14 @@ dirserv_generate_networkstatus_vote_obj(crypto_pk_t *private_key,
       smartlist_split_string(v3_out->net_params,
                              paramline->value, NULL, 0, 0);
     }
+
+    /* for transparency and visibility, include our current value of
+     * AuthDirMaxServersPerAddr in our consensus params. Once enough dir
+     * auths do this, external tools should be able to use that value to
+     * help understand which relays are allowed into the consensus. */
+    smartlist_add_asprintf(v3_out->net_params, "AuthDirMaxServersPerAddr=%d",
+                           d_options->AuthDirMaxServersPerAddr);
+
     smartlist_sort_strings(v3_out->net_params);
   }
   v3_out->bw_file_headers = bw_file_headers;

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.