Use "socket:/path/to/socket" syntax everywhere for AF_UNIX ports

[nickm]

With #11485, Andrea did a cool syntax for AF_UNIX hidden-service address targets.

Why didn't all the other FooPort options think of that?

We shouldn't have a separate option for Unix vs IP address as in:

SocksPort 127.0.0.1:9050
SocksSocket /home/nickm/.tor/connect-to-this

Instead why not just have it be:

SocksPort 127.0.0.1:9050
SocksPort socket:/home/nickm/.tor/connect-to-this

[dgoulet]

"socket" is a bit broad, UDP, Unix, or any of the 64 others network stack of the kernel? :)

What about this?

SocksPort unix:/home/nickm/.tor/connect-to-this

[dgoulet]

Ok, I lost control and implemented this :). There are two commits, one to implement the support and the other one to make hidden service use the same function to parse the port.

Let me know if that makes sense, see branch bug14451_026_v1.

FYI, we can litterally remove SocksSocket option with that and we should think about it before releasing 0.2.6 or else we end up supporting that option when SocksPort works with Unix socket...

[andrea]

This branch looks good to me.

[nickm]

I ran into an assertion failure when I did ORPort unix:/home/nickm/foo , so I fixed it by forbidding anything besides SocksPort and ControlPort.

I also removed SocksSocket and added documentation.

I've updated some new commits in my bug14451_026_v1 branch. Have a look at those?

[dgoulet]

Looks good! ACK.

[nickm]

Merged; thanks!