[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #22029 [Core Tor/Tor]: Allow ed25519 keys to be banned in the approved-routers file



#22029: Allow ed25519 keys to be banned in the approved-routers file
-------------------------------------------------+-------------------------
 Reporter:  teor                                 |          Owner:  neel
     Type:  enhancement                          |         Status:
                                                 |  needs_review
 Priority:  Medium                               |      Milestone:  Tor:
                                                 |  0.4.1.x-final
Component:  Core Tor/Tor                         |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  034-triage-20180328,                 |  Actual Points:
  034-removed-20180328                           |
Parent ID:                                       |         Points:  1
 Reviewer:  asn                                  |        Sponsor:
-------------------------------------------------+-------------------------

Comment (by teor):

 Replying to [comment:34 neel]:
 > I have created tests for the ed25519 and RSA functions. However, this
 did require moving some code from `process_descs.c` to `process_descs.h`
 so it can be used by the test.

 Thanks!

 I'd like to see a test for dirserv_load_fingerprint_file(), because the
 memory safety bugs were in that function. The existing test for
 dirserv_read_measured_bandwidth() is a good example of a unit test that
 reads a file.

 You force-pushed your changes over the old branch. When you did that,
 GitHub deleted my old review comments. I also can't tell which code I have
 already reviewed, and which code has been changed by your most recent
 changes. I don't have time to do a full review of your branch this week,
 so I'll have a look at it next week.

 In future, please add commits to the branch, rather than force-pushing
 your changes. When people review your tor pull requests, please add new
 commits for any pull request changes. Then reviewers won't have to do full
 reviews every time you make a change.

 If you want to change an existing commit, you can add fixup commits using:
 {{{
 git commit --fixup (existing commit hash)
 }}}
 Then we will squash before we merge,

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/22029#comment:35>
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