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