Opened 4 years ago

Closed 2 years ago

#12124 closed defect (wontfix)

globe/atlas don't display accept6/reject6 lines

Reported by: arma Owned by: cydrobolt
Priority: Medium Milestone:
Component: Metrics/Globe Version:
Severity: Normal Keywords:
Cc: phw, isis Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Child Tickets

Attachments (2)

atlas-bug_12124.jpg (190.7 KB) - added by karsten 4 years ago.
Screenshot of phw's bug_12124 branch.
atlas-bug_12124-2.jpg (191.2 KB) - added by karsten 4 years ago.
Screenshot of phw's bug_12124-2 branch.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 4 years ago by karsten

Cc: phw added

Here's what Onionoo knows about this relay's accept6/reject6 lines:

"exit_policy_v6_summary":{"accept":["20-23","43","53","79-81","88","110","143","194","220","389","443","464","531","543-544","554","563","636","706","749","873","902-904","981","989-995","1194","1220","1293","1500","1533","1677","1723","1755","1863","2082-2083","2086-2087","2095-2096","2102-2104","3128","3389","3690","4321","4643","5050","5190","5222-5223","5228","5900","6660-6669","6679","6697","8000","8008","8074","8080","8087-8088","8332-8333","8443","8888","9418","9999-10000","11371","12350","19294","19638","23456","33033","64738"]},

That's the same format as "exit_policy_summary" which is already displayed by Atlas and Globe. Shouldn't be too hard to also display "exit_policy_v6_summary".

Cc'ing phw for the Atlas side of things.

comment:2 Changed 4 years ago by phw

I have a fix in my branch bug_12124:
https://gitweb.torproject.org/user/phw/atlas.git/commitdiff/2f71621934d3c4b2365bf4a8e3d5109433d31c6f

It displays an additional IPv6-specific box. Is that usable enough?

Changed 4 years ago by karsten

Attachment: atlas-bug_12124.jpg added

Screenshot of phw's bug_12124 branch.

comment:3 Changed 4 years ago by karsten

I wonder, would it be more usable to have separate fields for "Exit Policy Summary (IPv4)" and "Exit Policy Summary (IPv6)"? (See this screenshot for the current version.)

comment:4 in reply to:  3 Changed 4 years ago by phw

Replying to karsten:

I wonder, would it be more usable to have separate fields for "Exit Policy Summary (IPv4)" and "Exit Policy Summary (IPv6)"? (See this screenshot for the current version.)

Yeah, I think it looks better. Updated in my bug_12124-2 branch:
https://gitweb.torproject.org/user/phw/atlas.git/commitdiff/refs/heads/bug_12124-2

Changed 4 years ago by karsten

Attachment: atlas-bug_12124-2.jpg added

Screenshot of phw's bug_12124-2 branch.

comment:5 Changed 4 years ago by karsten

Looks better! (See this screenshot.)

comment:6 Changed 4 years ago by phw

FYI, I merged the patch to Atlas' master and deployed it.

comment:7 Changed 4 years ago by karsten

Thanks. Leaving the ticket open for the same fix in Globe.

comment:8 Changed 3 years ago by isis

Owner: changed from rndm to isis
Status: newassigned

Changing Globe tickets to be owned by isis.

comment:9 Changed 3 years ago by isis

Status: assignednew

comment:10 Changed 3 years ago by cydrobolt

Owner: changed from isis to cydrobolt
Status: newassigned

comment:11 Changed 3 years ago by isis

Cc: isis added

FYI, if you change the ownership of a Globe ticket, please add isis to the CC line! Otherwise, I might not get notifications and notice that you've worked on something.

comment:12 Changed 2 years ago by karsten

Resolution: wontfix
Severity: Normal
Status: assignedclosed

We're about to retire Globe, and this issue is already resolved in Atlas, so closing as won't fix.

Note: See TracTickets for help on using tickets.