[tor-bugs] #17178 [Core Tor/Tor]: Rendezvous Single Onion Services: One-Hop Intro Point and Rendezvous

[Tor Bug Tracker & Wiki]

#17178: Rendezvous Single Onion Services: One-Hop Intro Point and Rendezvous
-------------------------------------------------+-------------------------
 Reporter:  teor                                 |          Owner:  teor
     Type:  enhancement                          |         Status:
                                                 |  needs_review
 Priority:  High                                 |      Milestone:  Tor:
                                                 |  0.2.9.x-final
Component:  Core Tor/Tor                         |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  rsos, tor-hs, TorCoreTeam201608,     |  Actual Points:  10
  review-group-5, review-group-7                 |
Parent ID:  #19923                               |         Points:  6.5
 Reviewer:  dgoulet                              |        Sponsor:
-------------------------------------------------+-------------------------

Comment (by dgoulet):

 On the overall patch, I found minor things:

 * Long tab :)
 {{{
 +    if (rend_service_use_direct_connection(options, rp)) {
 +          flags = flags | CIRCLAUNCH_ONEHOP_TUNNEL;
 +    }
 }}}

 * `rend_service_reveal_startup_time()` is exactly the same thing as
 `rend_service_allow_direct_connection()` so making the reveal startup time
 function use `rend_service_allow_direct_connection()` could be a better
 choice as the requirement for now is for the service to allow direct
 connection. I like the separation of semantic here so something like this
 would be good I think:

 {{{
 int rend_service_reveal_startup_time(const or_options_t *options)
 {
    return rend_service_allow_direct_connection(options);
 }
 }}}

 I'm happy with the rest! I'll leave this in `needs_review` for asn to take
 a look at it.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/17178#comment:65>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online