[tor-bugs] #14847 [Tor]: Controller: add a command to fetch HS descriptor from HSdir(s)

[Tor Bug Tracker & Wiki]

#14847: Controller: add a command to fetch HS descriptor from HSdir(s)
-----------------------------+----------------------------------------
     Reporter:  dgoulet      |      Owner:  dgoulet
         Type:  enhancement  |     Status:  needs_review
     Priority:  normal       |  Milestone:  Tor: 0.2.7.x-final
    Component:  Tor          |    Version:
   Resolution:               |   Keywords:  SponsorR tor-hs controller
Actual Points:               |  Parent ID:  #3521
       Points:               |
-----------------------------+----------------------------------------

Comment (by dgoulet):

 Replying to [comment:27 atagar]:
 > I've
 [https://gitweb.torproject.org/stem.git/commit/?id=162206da75b2eab2e8ab7b2866e70493d44d31d4
 added Stem support for HS_DESC_CONTENT events]. This helped highlight a
 few issues...
 >
 > 1. Events need to start with '650+HS_DESC_CONTENT' and end with a '650
 OK'. The [https://gist.github.com/DonnchaC/b76d934b8601ca9b3be0 example
 DonnchaC had] lacked both, causing Stem to treat it as a single line event
 (... which is actually precisely right according to the spec). For an
 example of what a multi-line event should look like see
 [http://paste.debian.net/plain/161282 this].
 >

 Fixed!

 > 2. The spec has 'HsDir = Fingerprint' but it's actually a LongName. Just
 need a spec tweak so we match HS_DESC: "HsDir = LongName / Fingerprint".
 >

 Fixed!

 > 3. When implementing this I assumed HSAddress will be 'UNKNOWN' if
 undefined (see earlier comment).
 >

 Yes, if the HSFETCH fails, the HS_DESC_CONTENT will print an "UNKNOWN".
 Fixed in control-spec/code.

 > Other than that looks good!

 You can find all the fixes in the following branches:

 Spec: `ticket14847_10`
 Code: `bug14847_027_03`

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