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

Re: [tor-bugs] #13125 [Tor]: Review the guardiness python script of #9321

#13125: Review the guardiness python script of #9321
------------------------+--------------------------------
     Reporter:  asn     |      Owner:
         Type:  defect  |     Status:  needs_review
     Priority:  normal  |  Milestone:  Tor: 0.2.6.x-final
    Component:  Tor     |    Version:
   Resolution:          |   Keywords:  tor-guard tor-auth
Actual Points:          |  Parent ID:  #9321
       Points:          |
------------------------+--------------------------------

Comment (by atagar):

 > Hmmmmm. I worry slightly about using stem for parsing here ... remind
 me, does stem accept unrecognized elements and fields and so forth in
 documents, or does it reject those? I thought it sometimes needed to get
 patched when we added new fields. If that's right, this will make us fail
 when the format evolves.

 Hi Nick. It's not clear to me what the spec change being proposed here is.
 To more generally answer your question though, new descriptor lines are
 fine for Stem as long as they don't violate the spec in some way. They're
 available via the
 [https://stem.torproject.org/api/descriptor/descriptor.html#stem.descriptor.__init__.Descriptor.get_unrecognized_lines
 get_unrecognized_lines()] method on the Descriptor base class.

 Cheers! -Damian

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/13125#comment:13>
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