Include Rack::TempFileReaper in Grape ?

[ericproulx]

I've noticed that tempfiles created when uploading a file (Rack::Multipart) are not closed and unlink by default. Fortunately, Rack provides a middleware for it and we just need to add it

Rails includes it by design so its not an issue when using Grape through it.

[hiSandog]

The scope would be easier to review if the first version defined a small acceptance contract for Include Rack::TempFileReaper in Grape ?. I would call out the default behavior, the migration/backwards-compatibility story, and one or two cases that are intentionally out of scope so the implementation does not grow beyond the original request.

[ericproulx]

I think having a opt-in parameter like lint! would be a great start.