[tor-bugs] #13339 [Tor]: Merge GSoC project - Consensus Diffs

[Tor Bug Tracker & Wiki]

#13339: Merge GSoC project - Consensus Diffs
-------------------------+-------------------------------------------------
     Reporter:  mvdan    |      Owner:
         Type:           |     Status:  needs_revision
  enhancement            |  Milestone:  Tor: 0.2.7.x-final
     Priority:  major    |    Version:  Tor: 0.2.7
    Component:  Tor      |   Keywords:  gsoc, merge, tor-client, prop140,
   Resolution:           |  027-triaged-1-in
Actual Points:           |  Parent ID:
       Points:  medium   |
-------------------------+-------------------------------------------------

Comment (by mvdan):

 I've addressed most of the comments:
 https://github.com/mvdan/tor/commits/consdiff

 The last commit adds four TODOs, which I think are the only things left to
 deal with.

 // TODO: change bool with max size to take up in disk

 I'll do this one.

 // TODO: check that the stored consensus is intact

 Not sure how to do this one yet.

 // TODO: duplicate code from add_networkstatus_bytes_to_outbuf, works but
 is
 // probably buggy.

 All of this code is confusing to me, so help from someone who initially
 wrote it would be great.

 // TODO: might want to do this in the background and have a lock to not
 // serve consensus diffs until they are updated

 What do we want to do about this one after all?

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