[tor-bugs] #30644 [Core Tor/Tor]: Add our own base-64 encoded ed25519 public key in dirserv_add_own_fingerprint()

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Jun 21 00:56:14 UTC 2019


#30644: Add our own base-64 encoded ed25519 public key in
dirserv_add_own_fingerprint()
--------------------------+------------------------------------
 Reporter:  teor          |          Owner:  neel
     Type:  enhancement   |         Status:  needs_revision
 Priority:  Medium        |      Milestone:  Tor: 0.4.2.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:  tor-ed25519   |  Actual Points:
Parent ID:  #22029        |         Points:
 Reviewer:                |        Sponsor:
--------------------------+------------------------------------

Comment (by teor):

 Replying to [comment:3 teor]:
 > The code needs these changes:
 > * add_rsa_fingerprint_to_dir() can fail and return -1, if it does,
 dirserv_add_own_fingerprint() should return -1
 > * add_ed25519_to_dir() can fail and return -1, if it does,
 dirserv_add_own_fingerprint() should return -1
 >
 > You can do the check and the warning like this:
 >
 https://github.com/torproject/tor/pull/970/commits/f1a236a8360ce0af2ad1bece4e89960463e57e3f
 #diff-c535b81e06d7fed27d5d3cdaabf1c4b8R160
 > But please use a different log message for each warning.

 The code should also have unit tests.

 Please remove all the #22029 commits from the branch: this branch should
 start with "Add our own base-64 encoded ed25519 public key in
 dirserv_add_own_fingerprint()" and any fix ups and tests.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30644#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list