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