#18910: distributing descriptors accross CollecTor instances
 Reporter:  iwakeh             |          Owner:  iwakeh
     Type:  enhancement        |         Status:  needs_review
 Priority:  High               |      Milestone:  CollecTor 1.1.0
Component:  Metrics/CollecTor  |        Version:
 Severity:  Normal             |     Resolution:
 Keywords:  ctip               |  Actual Points:
Parent ID:                     |         Points:
 Reviewer:                     |        Sponsor:

Comment (by iwakeh):

 Replying to [comment:47 karsten]:
 > Regarding the protocol, you're probably right that it's still not
 entirely correct.  To be honest, I'm having a hard time understanding the
 format there.  Would you mind looking at the test path changes above and
 adapting the protocol accordingly?  I can take another look at the
 protocol changes.

 Thanks for adding the test cases!
 I'll adapt the protocol after finishing the implementation changes and
 might add examples to clarify the definition.

 > Okay, after reading/skimming this ticket, I think it would make sense
 for me to review that second branch with all the fixes, but I only see 1
 commit there that is not yet in master.  Can you put together a branch
 with all the changes somewhere?  Or if that's not possible, can you point
 to specific commits that I should be looking at?

 Ok, just wait with the review for the completed changes.

Ticket URL: <https://trac.torproject.org/projects/tor/ticket/18910#comment:48>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
tor-bugs mailing list

Reply via email to