Opened 7 years ago

Closed 7 years ago

#6681 closed enhancement (implemented)

top defaults to 10 if URL parameter 'top' is not given

Reported by: cypherpunks Owned by: gsathya
Priority: Medium Milestone:
Component: Metrics/Compass Version:
Severity: Keywords:
Cc: karsten Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

https://compass.torproject.org/result?top=10
and
https://compass.torproject.org/result

result in the same output (10 lines).

Expected output, the follwing URL should show all relays:
https://compass.torproject.org/result

Child Tickets

Attachments (1)

0001-Change-default-value-of-top.patch (1023 bytes) - added by gsathya 7 years ago.

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by gsathya

comment:1 Changed 7 years ago by gsathya

Cc: karsten added
Owner: set to gsathya
Status: newassigned

https://compass.torproject.org/result?top=-1 gives you all the relays. I guess it makes sense to make it default to "show all relays". Added patch.

comment:2 Changed 7 years ago by gsathya

Status: assignedneeds_review

comment:3 Changed 7 years ago by karsten

Resolution: implemented
Status: needs_reviewclosed

Looks good to me. Merged with tiny tweaks. Closing.

Note: See TracTickets for help on using tickets.