[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #2553 [Tor Hidden Services]: tor2web mode for accessing hidden services
#2553: tor2web mode for accessing hidden services
---------------------------------+------------------------------------------
Reporter: arma | Owner: rransom
Type: enhancement | Status: needs_review
Priority: normal | Milestone:
Component: Tor Hidden Services | Version:
Keywords: | Parent: #2552
Points: | Actualpoints:
---------------------------------+------------------------------------------
Changes (by rransom):
* cc: hellais (added)
Comment:
Replying to [comment:7 nickm]:
> I think I like it!
>
> Some code issues you shouldn't have to fix:
> * I find the more complicated asserts hard to read. I'll refactor
them if we merge, though; my confusion is my own problem.
> * directory_initiate_command_routerstatus_rend()'s interface sure is
hideous, isn't it? We should do something about that. Not a problem
introduced by this patch, though.
>
> Also, the security implications of having a "don't be anonymous" mode
worry me some. Can we do more to make sure that no user ever thinks that
turning this on is a good idea?
A warning-level log message at startup and/or whenever the configuration
is modified/reloaded is probably appropriate.
> The check in connection_ap_rewrite_and_attach is a good start, but I
worry about accidentally breaking it later. Can we have this whole
feature be disabled unless the user supplies a compile-time option, for
instance?
Yes. The best place to put a `#ifdef` is in `src/or/config.c`; it should
require that Tor2webMode be unconfigured or off when the feature is not
enabled at compile time, and it should require that Tor2webMode be
explicitly turned on when the feature ''is'' enabled at compile time.
(Otherwise distribution packages might turn the compile-time flag on for
everyone, thus defeating its purpose.)
> (Is there any reason not to do that?)
I didn't do that because I don't understand GNU autotools.
> Also, have the tor2web people tried this out?
Yes. I'm CC-ing hellais, who says that he has tested this branch.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/2553#comment:8>
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