[tor-bugs] #3455 [Firefox Patch Issues]: Tor Browser should set SOCKS username for a request based on first party domain
Tor Bug Tracker & Wiki
blackhole at torproject.org
Wed Jul 16 09:10:54 UTC 2014
#3455: Tor Browser should set SOCKS username for a request based on first party
domain
-------------------------------------+-------------------------------------
Reporter: mikeperry | Owner: mikeperry
Type: enhancement | Status: needs_review
Priority: major | Milestone: TorBrowserBundle
Component: Firefox Patch | 2.3.x-stable
Issues | Version:
Resolution: | Keywords: tbb-linkability, tbb-
Actual Points: | usability, MikePerry201407R
Points: | Parent ID: #5752
-------------------------------------+-------------------------------------
Comment (by gk):
Replying to [comment:39 arthuredelstein]:
> Replying to [comment:38 gk]:
> > Re part1/3: What is the reasoning behind renaming ApplyFilter()? Can't
we just leave it? That would break less things e.g. in the accompanying
unittest: https://mxr.mozilla.org/mozilla-
esr24/source/netwerk/test/unit/test_protocolproxyservice.js
>
> It's true that the method
> {{{
> applyFilter: function(pps, uri, pi) {
> return pps.newProxyInfo("http", "localhost", 8080, 0, 10,
> pps.newProxyInfo("direct", "", -1, 0, 0,
null));
> }
> }}}
> doesn't break if we keep the name ApplyFilter, because the second
argument is ignored. But still, pedantically speaking, the second
argument's name should be changed to `channel`.
Yeah, I definitely think this unittest needs to get fixed as well (I am
okay doing it not in this bug but when we prepare things for Mozilla).
Another reason for keeping the original method's name is that it is more
specific than just "apply" giving a better context on what is going on...
One thing I forgot: we probably need new IIDs as we are changing
interfaces.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3455#comment:40>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list