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

Re: [tor-bugs] #12716 [meek]: Make meek-client-torbrowser take the firefox command as a parameter

#12716: Make meek-client-torbrowser take the firefox command as a parameter
------------------------+----------------------------
     Reporter:  dcf     |      Owner:  dcf
         Type:  defect  |     Status:  needs_revision
     Priority:  minor   |  Milestone:
    Component:  meek    |    Version:
   Resolution:          |   Keywords:
Actual Points:          |  Parent ID:
       Points:          |
------------------------+----------------------------

Comment (by infinity0):

 `readlink -f` should work instead of `realpath`. I've asked tor-qa about
 this.

 The Go options library does not allow "required options" because they are
 an oxymoron - they recommend that required options should instead be a
 positional argument - https://groups.google.com/forum/#!topic/golang-
 nuts/agYvRPNaBNA

 So I thought a default value of "meek-browser-helper" was the best option
 - it would work at least some of the time (as opposed to none of the
 time), and distros can still install their own /usr/bin/meek-browser-
 helper if they disagreed with the example script given.

 An alternative is to set the default value to "", and if this is not set
 by the user then give a custom error message like "either set --helper or
 use meek-client directly". Will do that in a commit tomorrow if you
 prefer.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/12716#comment:15>
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