danieleades / monzo-lib

monzo API client library, in pure rust
16 stars 4 forks source link

add some transport middleware in order to report progress of network tasks #1

Closed danieleades closed 3 years ago

danieleades commented 5 years ago

This is a bit of a big task. The current transport layer is using Reqwest, which has no concept of middleware. This makes it a little difficult to add something like progress reporting in a third-party app. This can be implemented using Reqwest's streaming API, but it feels like something that belongs in a different crate.

I would consider moving from Reqwest to Surf for this reason but i'm disinclined until it's a little more stable

danieleades commented 3 years ago

i'm going to close this. the network tasks performed by this crate are trivial anyway, so i don't believe this is required