[Whonix-devel] #14272 [Applications/Tor Launcher]: Make Tor Launcher work with Unix Domain Socket option

Wed Aug 31 21:28:58 CEST 2016

#14272: Make Tor Launcher work with Unix Domain Socket option
-------------------------------------------------+-------------------------
 Reporter:  gk                                   |          Owner:  brade
     Type:  enhancement                          |         Status:
                                                 |  needs_review
 Priority:  High                                 |      Milestone:
Component:  Applications/Tor Launcher            |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  tbb-security, TorBrowserTeam201608R  |  Actual Points:
Parent ID:  #14270                               |         Points:
 Reviewer:                                       |        Sponsor:
                                                 |  SponsorU
-------------------------------------------------+-------------------------

Comment (by mcs):

 Replying to [comment:10 gk]:
 > Two things (no need to create a new branch if 2. is moot):
 >
 > 1. In the commit description s/is is use/is in use/

 Oops. Thanks!

 > 2. Why do we have a new param for getTorFile()? Do we expect more calls
 with `aMustExist === false`? If not, why not just omit that param (and do
 something like `"control_socket" == aTorFileType` again) or at least
 rename it to something more descriptive than `aMustExist` (like
 `aControlSocket`)?

 Kathy and I like your suggestion. Here is a revised patch:
 https://gitweb.torproject.org/user/brade/tor-
 launcher.git/commit/?h=bug14272-02&id=8871259c966755233b134a5ddb2b4926539d25c6

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/14272#comment:11>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online