Opened 6 years ago

Closed 6 years ago

#11283 closed defect (fixed)

When displaying a bridge's hashed IP, point to an explanation?

Reported by: arma Owned by: rndm
Priority: Medium Milestone:
Component: Metrics/Globe Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

When a bridge operator looks up his bridge on globe, and sees 10.116.166.233, the first instinct is that something has gone wrong.

Since that's the primary (or at least most obvious) attribute that we scrub for bridges, maybe we should add a little link near there, with text like "Why is this not my IP address" or "How this address is anonymized" or similar. Ideally with far fewer words yet equally clear.

Child Tickets

Change History (8)

comment:1 Changed 6 years ago by karsten

Good point. How about taking out bridge IP addresses entirely? There's little value in having them in Onionoo, and almost zero value in having them in Globe. Globe could display the OR port and leave out the address part.

comment:2 Changed 6 years ago by arma

Great. Maybe write <anonymized> instead of the (misleading) IP address.

comment:3 Changed 6 years ago by arma

Or just leave it out.

comment:4 in reply to:  1 Changed 6 years ago by bastik

Replying to karsten:

Globe could display the OR port and leave out the address part.

It might be nice to see if is an IPv4 and/or an IPv6 bridge*. So I suggest displaying "IPvX:$port".

[*]for bridges from strangers. (For bridge operators it could be nice to see if both IPv4 and IPv6 get advertised correctly.)

comment:5 Changed 6 years ago by rndm

Here's a build that displays the IP version and port like karsten suggested:
http://globe.rndm.de/canary/index-11283.html

Example bridge:
http://globe.rndm.de/canary/index-11283.html#/bridge/D6F0BCE240C72842A54FE121FF838D62FC2A041B

comment:6 Changed 6 years ago by karsten

That's what bastik suggested, not I. But looks good to me! Thanks!

comment:7 Changed 6 years ago by rndm

I released ​https://github.com/makepanic/globe/releases/tag/v0.4.6 which includes the change to display the port and ip version.
If this fixes the issue, feel free to close this.

comment:8 Changed 6 years ago by karsten

Resolution: fixed
Status: newclosed

I'd say this fixes the issue. If anyone disagrees, please re-open! Closing.

Note: See TracTickets for help on using tickets.