[tor-bugs] #8036 [Stem]: Tweak Stem's documentation

[Tor Bug Tracker & Wiki]

#8036: Tweak Stem's documentation
--------------------+-------------------------------------------------------
 Reporter:  amj703  |          Owner:  atagar
     Type:  defect  |         Status:  new   
 Priority:  normal  |      Milestone:        
Component:  Stem    |        Version:        
 Keywords:          |         Parent:        
   Points:          |   Actualpoints:        
--------------------+-------------------------------------------------------

Comment(by atagar):

 > See the attached patch. Please tweak as you see fit.

 Thanks, pushed.

 > I know. I wrote that script above. :)

 Ahhhh. :P

 > Is b2a_hex and a2b_base64 what you'd use, too, or are there other
 methods that you prefer?

 No, that's what I'd use. The '=' padding worries me a bit (can we be sure
 that we'll always need to pad by two?), but otherwise looks good.

 > Are you fine with making sure that all hex strings are upper-case?

 Yup. We should standardize on something and most are uppercase, so I agree
 that we should use that.

 > IPv4 addresses are okay, too. The current Tor code doesn't have IPv4
 addresses in a or or-address lines, but that can be added at any point.

 Oh well. Changed...

 https://gitweb.torproject.org/stem.git/commitdiff/c6a9cde0f4eb5c627f5ba41f9d263be0c5854ae8

 > What is the second "it" in your sentence?

 See the spec change that I cited:
 https://gitweb.torproject.org/torspec.git/commitdiff/e2aafe8

 ... it says consensus-method 14 everywhere except once.

 Cheers! -Damian

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