[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #30255 [Core Tor/sbws]: Add additional bandwidth file headers in sbws 1.2



#30255: Add additional bandwidth file headers in sbws 1.2
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  task           |         Status:  new
 Priority:  Medium         |      Milestone:  sbws: 1.2.x-final
Component:  Core Tor/sbws  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+-----------------------------------

Comment (by karsten):

 irl and I discussed the topic of additional bandwidth headers as part of
 #30216, where we're trying to write a parser for these files.

 When reading this ticket and its subtickets I'm slightly concerned that
 you might consider adding spaces to new header lines. For example,
 operating system and contact information sound like they might contain
 spaces. But as far as I see it, that means trouble! One reason is that
 `bandwidth-file-headers` in dir-spec doesn't permit spaces in `KeyValue`
 parts, so that it wouldn't be possible to copy over header lines to the
 vote.

 Does this make sense?

 In fact, if it does, would you mind clarifying in the spec that header
 lines always have the format `HeaderLine ::= KeyValue NL`? That would also
 simplify the parser, because we'd then be able to distinguish header and
 relay lines by the number of space-separated `KeyValue` elements.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30255#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs