Changes between Initial Version and Version 1 of Ticket #22488, comment 23


Ignore:
Timestamp:
Jan 18, 2018, 12:44:10 PM (5 weeks ago)
Author:
teor
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #22488, comment 23

    initial v1  
    22> Replying to [comment:21 teor]:
    33> > * clarify that the recommended version uses the consensus version at https://metrics.torproject.org/onionoo.html#details_relay_recommended_version
    4 
    5 Replying to [comment:21 teor]:
    6 > ...
    7 > * clarify that the recommended version uses the consensus version at https://metrics.torproject.org/onionoo.html#details_relay_recommended_version
    84
    95Please see my branch bug22488 at https://github.com/teor2345/metrics-web.git
     
    1410> Yes, it would be great if onionoo includes both and atlas could then display a clear info when they are not the same (and display only one if they are the same, so to not confuse operators).
    1511
    16 > * make Onionoo parse the platform field in the descriptor and compare it to the consensus version
    17 
    1812I am not sure if this fix is a good idea. We don't trust descriptors when they are not in a consensus. (Their details can be wrong.) So why are any of our tools using them? And what should Onionoo do if a relay has multiple descriptors?
    1913
     
    2115
    2216For an alternative fix where Onionoo stops parsing untrusted descriptors, see #24932.
     17
     18Edit: remove duplicate quotes