stefan-mysten
stefan-mysten
I think we could use `RequestBodyLimit` layer from `tower` and add it in our route builder, but unfortunately in this version it requires the request to be of type `http_body::Limited>`...
> Thanks for looking into this @stefan-mysten -- echoing our conversation elsewhere: Using `Content-Length` is a great idea -- and doing that by adding middleware to capture the content length,...
Refactored the code. This is hands down the worst code I ever wrote ðŸ˜
@amnn I addressed all comments except the more tests one. Will try to get to it in the morning, but I might not be able to.
Thanks @amnn and @wlmyng for finishing this PR! 🙇
Thanks @vegetabledogdog! I think this is reasonable.
@tzakian what do you think?
Sorry for missing this PR @poelzi!
Can you explain what do you mean by size: ```0xf699e7f2276f5c9a75944b37a0c5b5d9ddfd2471bf6242483b03ab2887d198d0, both say the size is 4066```? You mean the number of dynamic fields that this object has?
@bmwill Lots of updates here * got rid of `safer_ffi` * added more tests * finished the vector implementation * added more docs Still needs to be decided: which types...