Relays that aren't Valid never get Running
We use router_is_active() for too many checks when directory authorities are deciding how to handle relays that don't have the Valid flag.
Once upon a time, you could be missing a Valid flag and still get the Running flag. That would cause clients to avoid using you except in circuit positions specified in their 'AllowInvalidRelays' config option.
At present if we take away your Valid flag, we also necessarily take away your Running flag.
We should sort out what we want to do. I think there is still a role for having "dangerous" relays -- meaning you don't use them at the beginning or the end of your path.
Maybe this means we should do away with the 'Valid' flag, and add a !badguard along with !badexit?
[Automatically added by flyspray2trac: Operating System: All]