[tor-bugs] #17668 [Tor]: moria1, with updated v3 cert: Bug: Generated a networkstatus consensus we couldn't parse.

[Tor Bug Tracker & Wiki]

#17668: moria1, with updated v3 cert: Bug: Generated a networkstatus consensus we
couldn't parse.
-------------------------------------------------+-------------------------
 Reporter:  arma                                 |          Owner:  nickm
     Type:  defect                               |         Status:
 Priority:  High                                 |  needs_review
Component:  Tor                                  |      Milestone:  Tor:
 Severity:  Blocker                              |  0.2.8.x-final
 Keywords:  must-fix-before-028-rc,              |        Version:  Tor:
  027-backport, needs-merge                      |  0.2.7
Parent ID:                                       |     Resolution:
 Reviewer:                                       |  Actual Points:
                                                 |         Points:
                                                 |        Sponsor:
-------------------------------------------------+-------------------------
Changes (by teor):

 * keywords:  must-fix-before-028-rc, 027-backport => must-fix-
     before-028-rc, 027-backport, needs-merge
 * version:   => Tor: 0.2.7


Comment:

 I'm happy with T1/S1 for torspec, and S2, T2, T4/T5, and T6 for tor.

 T3's code for tor is ok, but we've changed the behaviour, so let's update
 the function comment:
 {{{
 /** If there are entries in <b>routers</b> with exactly the same ed25519
 keys,
  * remove the older one. If they are exactly the same age, remove the one
 with
  * the greater descriptor digest. May alter the order of the list. */
 }}}

 After that, let's get it merged. (No need to re-review comment-only
 changes.)

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