restly icon indicating copy to clipboard operation
restly copied to clipboard

Bugfix in routes.js 404; Support for reading request body

Open dwelch2344 opened this issue 8 years ago • 1 comments

, instead of . - could happen to anyone ;)

dwelch2344 avatar May 09 '17 04:05 dwelch2344

I've been playing around with Restly and really like it. I saw the comment on #8 and figured I'd provide a solution I whipped together.

I'm basically adding a third (and thus optional) context param to routes that contains both the request (req) and response (res). I also had to add the express JSON bodyParser, but that should have little-to-no impact on the rest of the framework.

Happy to debate the impl and take a different approach if necessary. Thoughts?

dwelch2344 avatar May 09 '17 17:05 dwelch2344