search

bellycard / napa

A simple framework for building APIs with Grape
Other
329 stars 72 forks source link

add missing require for param sanitizer #258

Closed ashtonthomas closed 7 years ago

ashtonthomas commented 8 years ago

@bellycard/platform

We need this require for hitting this code path outside of an application

It was added in the main codebase but we don't have that in actionpack 3.2.0

I need this for https://github.com/bellycard/napa/pull/257 as this spec hits this code path and errors.

ashtonthomas commented 8 years ago

bump

itsbagpack commented 8 years ago

Did you test that this worked for both apps that were using actionpack < 3.2.0 and actionpack > 4.0 ?

If so, +1

ashtonthomas commented 8 years ago

@ywong19, thanks! I tested i some, but I will test in all of those cases