We removed this property in T348659: [S] Decide which tunnels to store because at the time we had no use for it. However, T380711: Provide ip_reputation_tunnel_type AbuseFilter variable will use it to determine whether or not to return data so we should re-implement it.
Additionally, that means null will no longer be a valid entry so we should probably write a maintenance script that periodically clears out these stale values.
I think this should be a 3 step process:
- record the anonymous property, ensure that it's being correctly written to the tunnels table
- audit and wherever necessary, update calls to the tunnel table. We don't reveal the anonymous property anywhere (because we don't expect it) but that doesn't guarantee the queries in use now will still work after this update
- implement the maintenance script to clear out now-invalid rows in tunnels