Changes between Initial Version and Version 1 of Ticket #23817, comment 54


Ignore:
Timestamp:
Nov 20, 2017, 4:41:15 PM (3 years ago)
Author:
asn
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #23817, comment 54

    initial v1  
    99`bug23817_usable_filtered_033_v2` and `bug23817_usable_filtered_032_v2` which should be the minimal change here.
    1010
     11The way I see it, moving from checking filtered guards to checking reachable filtered guards, moves us from overblocking to underblocking which seems better and less disturbing. That is, when we checked filtered guards we could blacklist guards even tho most of them don't actually work (like teor's example in comment:47). Now that we check reachable filtered guards we could potentially not blacklist guards when we actually should (e.g. if the network was down, and we marked all our guards as unreachable, and then one guard failed to give us an md, we will not blacklist it just because the other guards were found unreachable).
     12
    1113Let me know how you feel about this one.