[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #30644 [Core Tor/Tor]: Add our own base-64 encoded ed25519 public key in dirserv_add_own_fingerprint()
#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
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs