Christian Haynes
Christian Haynes
Adding a ureq client as part of PR #116 but would love some feedback
Interesting, thanks for testing this! I will look into it and see what's up, I'm likely doing something wrong somewhere.
I recreated the issue and found the tweak, pretty simple fix really. Where I am using `read_to_string()` I needed to be using `read_to_vec()`, just slipped past me. I will get...
I think the problem is we have to clone the response and that was the issue. There might be a way around this but I'm not sure what that would...
I might have confused this with the request being cloned, I will look into this and play around with it. In the meantime if you already have a fix in...
Nice! I will absolutely check this out
I have (hopefully) created a streaming implementation in #115 though I would love more testing and eyes on it as this was a first for me
I could see this being useful, though I think it might be worth wrapping the value in an Option
> I wrapped the metadata in an Option, i agree it's better in term of DX, also updated underlying library to the new process_response API. > > At the moment...
I think we could do something like this: ``` Add a metadata callback to HttpCacheOptions pub type MetadataProvider = Arc< dyn Fn(&request::Parts, &response::Parts) -> Option + Send + Sync >;...