Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#18826 closed enhancement (fixed)

Allow setting the listen address for all Chutney hosts

Reported by: anonym Owned by: anonym
Priority: Medium Milestone:
Component: Core Tor/Chutney Version:
Severity: Normal Keywords: tor-chutney-usability
Cc: Actual Points:
Parent ID: Points: .1
Reviewer: Sponsor: SponsorS-can

Description

In Tails' automated test suite we have plans to use Chutney to run a simulated Tor network (for increased performance and, in particular, robustness). Of course, we do not want to run that Chutney instance in the system under testing (i.e. the Tails instance) but on the host orchestrating the tests. In order to avoid having to forward traffic via firewall rules, we'd like it to be possible to make the Chutney hosts listen on some other IP address than localhost.

The attached patch allows setting the address all Chutney hosts will listen on via the CHUTNEY_LISTEN_ADDRESS environment variable.

Child Tickets

Attachments (1)

0001-Allow-specifying-the-Chutney-listen-address.patch (4.1 KB) - added by anonym 4 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 4 years ago by teor

Status: newneeds_review

I think this might come under one of our sponsors (at least under "can"), but I don't know which one.

comment:2 Changed 4 years ago by anonym

I apparently submitted the wrong patch -- the new one is more complete and e.g. verify still works unlike in the first patch. Sorry for the confusion!

comment:3 Changed 4 years ago by teor

Sponsor: SponsorS-can

comment:4 Changed 3 years ago by nickm

Keywords: tor-chutney-usability added

comment:5 Changed 3 years ago by nickm

Points: .1

comment:6 Changed 3 years ago by nickm

Owner: changed from nickm to anonym
Status: needs_reviewassigned

comment:7 Changed 3 years ago by nickm

Status: assignedneeds_review

comment:8 Changed 3 years ago by teor

Resolution: fixed
Status: needs_reviewclosed

Merged to chutney master as 83ca28c with the following changes:

  • ignore the first part of the patch, it was obsoleted by 8e4f5cb: Do not use the system torrc when generating router keys
  • fix a typo in the commit message
  • ignore whitespace differences in favour of master, mainly in diff context

Tested on OS X, couldn't test on Linux because my box is running a bit too much at the moment.

comment:9 Changed 3 years ago by cypherpunks

I opened #19533 for documenting the new environment variable.

Note: See TracTickets for help on using tickets.