refactor to use should_record_bridge_info() more
We have this function
should_record_bridge_info
which simply checks
options->BridgeRelay && options->BridgeRecordUsageByCountry.
But in geoip_note_client_seen(), where we just added the DoS mitigation stuff, we don't use the function, instead choosing to check the variables directly. We make the same choice in the options_need_geoip_info() function.
We should refactor things so we use the function in all cases.
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information