Opened 9 months ago
Last modified 9 months ago
#29721 new defect
Refactor RelayList
Reported by: | juga | Owned by: | |
---|---|---|---|
Priority: | Medium | Milestone: | sbws: unspecified |
Component: | Core Tor/sbws | Version: | sbws: 1.0.5 |
Severity: | Normal | Keywords: | tech-debt, refactor, scanner, relays |
Cc: | juga | Actual Points: | |
Parent ID: | #28684 | Points: | |
Reviewer: | Sponsor: |
Description
Refactor 1:
It should probably have a dictionary of relays instead of a list, so that it is not needed to convert the list to dictionary in resultdump.
It should be then named Relays
or RelayDict
In https://github.com/torproject/sbws/issues/219 it was proposed to change the list to a set, but that's only used to exclude authorities in the prioritizer, which should also be a method in RelayList.
Child Tickets
Change History (2)
comment:1 Changed 9 months ago by
Keywords: | tech-debt refactor added |
---|
comment:2 Changed 9 months ago by
Keywords: | scanner relays added |
---|
Note: See
TracTickets for help on using
tickets.
Add keywords