Opened 10 years ago

Last modified 2 years ago

#2966 assigned enhancement

Include bridge country codes in sanitized bridge descriptors

Reported by: karsten Owned by: metrics-team
Priority: Low Milestone:
Component: Metrics/CollecTor Version:
Severity: Normal Keywords: metrics-geoip
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description (last modified by iwakeh)

Before May 2010, we resolved bridge IP addresses to country codes in the sanitizing process and appended them to the nickname, e.g., UnnamedDE. This was possible, because we sanitized descriptors in batches which was a semi-automatic process.

When we switched to sanitizing descriptors automatically, we took country codes out, because the code was too complex and broke all the time. The difficulties were that descriptors can arrive in any order (while we still need to come up with the same set of sanitized descriptors) and that we need to update the GeoIP database periodically. That's not impossible to solve, but also not easy to get right.

We should find a way to make this work, maybe by relaxing the requirement to update the GeoIP database at all. The old code that we took out in May 2010 is here.

Child Tickets

Change History (7)

comment:1 Changed 10 years ago by karsten

Owner: karsten deleted
Status: newassigned

comment:2 Changed 4 years ago by iwakeh

Severity: Normal
Status: assignedneeds_information

Is this still relevant? What's left to be done?

comment:3 Changed 4 years ago by karsten

It's still something that I'd want to have, because statistics on bridges by country seem quite interesting. But I still don't have a good design in mind how to build this. Hmmm.

comment:4 Changed 4 years ago by iwakeh

Description: modified (diff)
Milestone: CollecTor 1.1.0

Corrected git link in description.

Added to milestone 1.1.0

comment:5 Changed 4 years ago by karsten

Milestone: CollecTor 1.1.0

Unassigned from milestone, because of all the uncertainty with getting a good design for this.

comment:6 Changed 3 years ago by karsten

Owner: set to metrics-team
Status: needs_informationassigned

comment:7 Changed 2 years ago by irl

Keywords: metrics-geoip added
Note: See TracTickets for help on using tickets.