[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 karsten):

 Replying to [comment:7 atagar]:
 > > 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.

 We know the expected number of bits for these fields.  I'll take another
 look at base64 to make sure we need exactly two ='s.

 > > 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.

 Okay.  I'll prepare and test a patch.

 > > 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.

 Looks like this has already been fixed in
 [https://gitweb.torproject.org/torspec.git/commitdiff/5ddddcb 5ddddcb].

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