#28450: Add a parse_bytes function to load descriptors from bytes ---------------------------+----------------------------------- Reporter: irl | Owner: irl Type: enhancement | Status: needs_information Priority: Medium | Milestone: Component: Core Tor/Stem | Version: Severity: Normal | Resolution: Keywords: | Actual Points: Parent ID: | Points: Reviewer: atagar | Sponsor: ---------------------------+----------------------------------- Changes (by atagar):
* status: needs_review => needs_information Comment: Thanks irl! I took this in a slightly different direction. https://gitweb.torproject.org/stem.git/commit/?id=670f403 Changes from your patch are... * Moved this to Descriptor.from_str() to match the convention used elsewhere in stem's codebase. * Fixed a bug where the kwargs weren't passed along. * The method provides a single descriptor by default, with a **multiple = True** argument that must be supplied if you want a list instead. I did this because for the common case of parsing a single descriptor stem's been a little clunky. * Descriptor type inference can now be done in three ways: a **descriptor_type** argument, a **@type** annotation, and use of from_str() on subclasses (for example, **RelayServerDescriptor.from_str()**). * Test coverage. Thoughts? -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/28450#comment:3> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs