Opened 5 years ago

Closed 5 years ago

#11008 closed defect (fixed)

Accept Non-Tor Platform line in Server Descriptor

Reported by: sysrqb Owned by: atagar
Priority: Medium Milestone:
Component: Core Tor/Stem Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Stem currently assumes the platform line in the server descriptor starts with "platform Tor".

platform_match = re.match("^Tor (\S*).* on (.*)$", value)

There exist other implementations which say something else, such as "platform node-Tor". Maybe changing "^Tor" to "(\w*)" will work, but there could be a better way.

Child Tickets

Change History (2)

comment:1 Changed 5 years ago by sysrqb

There are instances of node-Tor that exist in the wild. For example, https://atlas.torproject.org/#details/E0671CF9CB593F27CD389CD4DD819BF9448EA834.

comment:2 Changed 5 years ago by atagar

Resolution: fixed
Status: newclosed

Added support for node-Tor relays. Thanks for the catch!

https://gitweb.torproject.org/stem.git/commitdiff/1f2ea0f

Note: See TracTickets for help on using tickets.