zacharygolba / via

A multi-threaded async web framework for people who appreciate simplicity.
Apache License 2.0
1 stars 0 forks source link

refactor(via): prefer http-body-util types #46

Closed zacharygolba closed 1 week ago

zacharygolba commented 2 weeks ago

An experimental refactor for a rainy day.

zacharygolba commented 2 weeks ago

I'm unsure how I feel about this. The http-body-util crate is amazing and well-designed to the point that it's hard to not want to use it. This PR definitely makes the code easier to maintain and reason about. However, I like the idea of people being able to see how their data moves throughout an application without having to dig through the docs of a peer dependency or reading rather complex macros. There are some improvements here independent of the move to http-body-util. If nothing else, some of the design improvements and compatibility with the http-body-util crate will make it upstream.