[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #28692 [Core Tor/sbws]: sbws should set ConnectionPadding 0



#28692: sbws should set ConnectionPadding 0
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  defect         |         Status:  needs_revision
 Priority:  Medium         |      Milestone:  sbws: 1.0.x-final
Component:  Core Tor/sbws  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:  teor           |        Sponsor:
---------------------------+-----------------------------------
Changes (by teor):

 * status:  needs_review => needs_revision
 * reviewer:   => teor


Comment:

 This patch makes the torrc options in sbws even more complicated.

 Here's what sbws has right now:
 1. a list of hard-coded torrc options in globals.py:
 https://github.com/torproject/sbws/blob/master/sbws/globals.py#L10
 2. a list of configured torrc options in stem.py, and the hard-coded
 option `LearnCircuitBuildTimeout 0`:
 https://github.com/torproject/sbws/blob/master/sbws/util/stem.py#L127
 3. a custom list of torrc options that the user can configure for launch:
 https://github.com/torproject/sbws/blob/master/sbws/util/stem.py#L142
 4. a list of hard-coded torrc options in stem.py, that can't be set until
 after sbws has bootstrapped:
 https://github.com/torproject/sbws/blob/master/sbws/util/stem.py#L197

 Then the pull request adds custom code to ignore the ConnectionPadding
 option if tor doesn't support it. This code is more complicated than it
 needs to be, and it has bugs.

 Here's a simpler design:
 * a list of torrc options in stem.py, containing 1., 2. and 4., with a
 flag saying when the option should be applied to tor (1. and 2. on launch,
 4. after launch)
 * a custom list of torrc options that the user can configure for launch
 (same as 3.)

 (There's an even simpler design in #28712 that makes 3. into a torrc file,
 but it's a breaking change.)

 Once we refactor the torrc option code, we can add:
 * a flag to ignore failures when applying the option after launch
 * a torrc option table entry: ConnectionPadding, 0, apply after launch,
 ignore failures

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/28692#comment:4>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs