Opened 7 years ago

Last modified 2 years ago

#5825 new defect

Bridges without geoip file report empty statistics

Reported by: karsten Owned by:
Priority: Low Milestone: Tor: unspecified
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-bridge metrics statistics geoip
Cc: nickm Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

A bridge that doesn't have a geoip file reports empty statistics, implying that it saw zero users. That's confusing, because we cannot distinguish this case from a bridge that actually didn't see any users. The bridge should simply leave out its statistics if it doesn't have a geoip file.

There are not many bridges affected by this bug, but we should probably fix it anyway. 0.2.3.x or 0.2.4.x?

Child Tickets

Change History (11)

comment:1 Changed 7 years ago by nickm

Milestone: Tor: 0.2.3.x-final
Priority: normalminor

Calling this 0.2.3.x, assuming that the fix is small and easy, but it would be okay to let it slip to 0.2.4.x IMO.

comment:2 Changed 7 years ago by nickm

Milestone: Tor: 0.2.3.x-finalTor: 0.2.4.x-final

comment:3 Changed 7 years ago by nickm

Keywords: tor-bridge added

comment:4 Changed 7 years ago by nickm

Component: Tor BridgeTor

comment:5 Changed 7 years ago by nickm

Keywords: 024-deferrable added

comment:6 Changed 7 years ago by nickm

Keywords: 024-deferrable removed
Milestone: Tor: 0.2.4.x-finalTor: 0.2.5.x-final

Deferring to 0.2.5, since validate_bridge_stats would reject this, and validate_bridge_stats looks pretty fragile and untested.

comment:7 Changed 6 years ago by nickm

Milestone: Tor: 0.2.5.x-finalTor: 0.2.???

comment:8 Changed 3 years ago by teor

Milestone: Tor: 0.2.???Tor: 0.3.???

Milestone renamed

comment:9 Changed 3 years ago by nickm

Keywords: tor-03-unspecified-201612 added
Milestone: Tor: 0.3.???Tor: unspecified

Finally admitting that 0.3.??? was a euphemism for Tor: unspecified all along.

comment:10 Changed 2 years ago by nickm

Keywords: tor-03-unspecified-201612 removed

Remove an old triaging keyword.

comment:11 Changed 2 years ago by nickm

Keywords: metrics statistics geoip added
Severity: Normal
Note: See TracTickets for help on using tickets.