Opened 7 months ago

Last modified 6 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 7 months ago by juga

Keywords: tech-debt refactor added

Add keywords

comment:2 Changed 6 months ago by juga

Keywords: scanner relays added
Note: See TracTickets for help on using tickets.