SocksListenAddress is deprecated but still used in default/sample torrc

This leads to problems such as: "SocksListenAddress can't be used when there are multiple SocksPort lines"

(that's using "experimental" deb repo)

changed milestone to %Tor: 0.2.3.x-final

added component::core tor/tor milestone::Tor: 0.2.3.x-final priority::high resolution::fixed status::closed tor-client type::defect version::tor 0.2.3.12-alpha labels

Trac:
Milestone: N/A to Tor: 0.2.3.x-final
Component: - Select a component to Tor Client

Ok. For reference the current torrc.sample stanza is:

## Replace this with "SocksPort 0" if you plan to run Tor only as a
## relay, and not make any local application connections yourself.
SocksPort 9050 # what port to open for local application connections
SocksListenAddress 127.0.0.1 # accept connections only from localhost
#SocksListenAddress 192.168.0.1:9100 # listen on this IP:port also

What's the new behavior we want? Perhaps something like

## What address/port to open for local application connections.
## Replace this with "SocksPort 0" if you plan to run Tor only as a
## relay, and not make any local application connections yourself.
SocksPort 9050 # binds to localhost if address isn't specified
#SocksPort 192.168.0.1:9100 # listen on this IP:port also

I don't think we should have "socksport 127.0.0.1:9050" as default-on in the torrc file, because that limits us in the future about where we want it to bind. Do we want 9050 to be enabled by default in the torrc at all? It already is the default in the code -- see calls to parse_port_config().

Hm.

Trac:
Priority: minor to major

I think arma's suggestion is fine; unless we get a better one, let's go with that.

I pushed a branch "bug5438" to my public repo that does approximately that change, and removes the rest of the *ListenAddress options for torrc.sample too. Please review?

Trac:
Status: new to needs_review

Looks like this got merged in 0.2.3.14-alpha.

Trac:
Status: needs_review to closed
Resolution: N/A to fixed

As an addendum: we also removed the "SocksPort 9050" line, which might confuse users into thinking that Tor won't default to it. I guess we'll find out.

(It is now the case that torrc.sample has no uncommented lines.)

Trac:
Keywords: N/A deleted, tor-client added

Trac:
Component: Tor Client to Tor

closed

moved to tpo/core/tor#5438 (closed)