Add a parse_bytes function to load descriptors from bytes

This is a convenience function that wraps the bytes in a BytesIO and calls parse_file().

added component::archived/stem owner::irl priority::medium resolution::implemented reviewer::atagar severity::normal status::closed type::enhancement labels

Trac:
0001-Adds-a-parse_bytes-function-to-load-descriptors.patch

Trac:
Status: new to accepted
Owner: atagar to irl

Trac:
Reviewer: N/A to atagar
Status: accepted to needs_review

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?

Trac:
Status: needs_review to needs_information

Resolving. Feel free to reopen if ya need anything else.

Trac:
Resolution: N/A to implemented
Status: needs_information to closed

I am now using this in bushel and it is working great.

closed