[tor-bugs] #21059 [Core Tor/Tor]: shared-rand-current-value violates spec

[Tor Bug Tracker & Wiki]

#21059: shared-rand-current-value violates spec
--------------------------+------------------------------------
 Reporter:  atagar        |          Owner:  asn
     Type:  defect        |         Status:  needs_review
 Priority:  High          |      Milestone:  Tor: 0.3.0.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Major         |     Resolution:
 Keywords:                |  Actual Points:
Parent ID:                |         Points:  0.1
 Reviewer:                |        Sponsor:
--------------------------+------------------------------------

Comment (by atagar):

 Hmmmm. The sentence that reads...

 "Each consensus method specifies an order in which these items MUST occur"

 ... strikes me as the same as what we previously had...

 "The preamble contains the following items.  They MUST occur in the order
 given here:"

 When I read 'each consensus method specifies an order' I think 'ok, so
 this is the order'. My change (simply changing MUST to SHOULD) drops any
 strong assurance that what follows is the actual ordering.

 That said, I don't care too strongly about this. This patch feels like a
 lateral move to me. If Nick prefers it I don't have a strong objection to
 merging it.

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