Barret Schloerke

Results 176 comments of Barret Schloerke

There has been a lot of work in https://github.com/rstudio/plumber/blob/06e46f3ff5119e5f1cb8af29ef49aecb3cbb932a/R/parse-query.R#L141 Currently only the path params are being coerced: https://github.com/rstudio/plumber/blob/76cecc58925897c91cb88203ab485a0c512a0d26/R/plumber-step.R#L278-L280 Will prolly need to parse `req$argsQuery `/`req$args` and `req$argsBody` as well: https://github.com/rstudio/plumber/blob/06e46f3ff5119e5f1cb8af29ef49aecb3cbb932a/R/plumber.R#L697-L702...

> I've gone for opt-in - if that's a breaking change, then embrace your inner Hadley and apply a bigger bump to the pacakge version number Opt-in is perfect. Maybe...

Related: https://github.com/analythium/tryr cc @gadenbuie

@slodge This provides a way forward without adding new serializer methods... * `req` and `res` should be added to `dev_on()` and `dev_off()` * `dev_on()` should accept `req` and `res`. *...

@JosiahParry Check out `vignettes/annotations.Rmd`. Thank you!

@JosiahParry I believe this PR has it done: https://github.com/rstudio/plumber/pull/905 . I'll start carving away time to get this through.

Locking old thread. If you have a reprex API that you can show has a memory leak, please open a new issue referencing this one. Thank you

Closing this issue in favor of solution provided by Dmitry in https://github.com/rstudio/plumber/issues/496#issuecomment-541402503 It seems really fair that the standard memory leak is due to a inefficient malloc linux library. Changing...