[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