Opened 8 months ago

Closed 4 months ago

#32264 closed task (fixed)

MS: Perform fetches of check.tpo in an exitmap plugin

Reported by: irl Owned by: irl
Priority: Medium Milestone:
Component: Metrics/Exit Scanner Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: #29654 Points:
Reviewer: karsten Sponsor:

Description

For this milestone, the connection helper (#32263) is glued to the ExitSpider class (#32262) in a new ExitScanner plugin.

It will be necessary to implement some parser for check.tpo's output in order to extract the IP address from it.

Child Tickets

Change History (5)

comment:1 Changed 7 months ago by irl

Summary: MS: Perform fetches of check.tpo in an ExitScanner pluginMS: Perform fetches of check.tpo in an exitmap plugin

exitmap can do this

comment:2 Changed 7 months ago by irl

Owner: changed from metrics-team to irl
Status: newaccepted

comment:3 Changed 7 months ago by irl

Reviewer: karsten
Status: acceptedneeds_review

https://github.com/irl/exitmap/commit/fb0e74e6f818aa368f5119f9499d84019ab6c52e is the commit, but really it is just adding a more verbose log message at a higher log level on line 73.

I think the big changes between this and the current system are:

  • things will be scanned more often (with the current network we can check every relay every 50 minutes or so)
  • there are no retries (maybe doesn't matter with the more frequent scanning, i'm not sure if the old system had retries)
  • the scanner is stateless, tor holds the state, it's the job of the merger script in #32265 to carry forward old results
  • the scanner doesn't output a published or last

Exit policies are checked to see if it's possible to exit to check.torproject.org. If it's not possible, no attempt will be made.

No descriptor information other than the fingerprint is in the output. This means exit lists don't contain the OR IP address. I /think/ Exonerator gets this from parsing server descriptors anyway.

Review is more for the concept than the code.

comment:4 Changed 5 months ago by irl

Status: needs_reviewaccepted

comment:5 Changed 4 months ago by irl

Resolution: fixed
Status: acceptedclosed

This is completed.

Note: See TracTickets for help on using tickets.