New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

> added option to provide bodyParse limit #11

Merged

bengourley merged 1 commit into master from bug/body-parse-size-option

Mar 24, 2016

Contributor

bag-man commented Mar 24, 2016

@bengourley Could you QA this for the WSJ ticket?

Adam has looked over it, so hopefully everything is in order.


          > added option to provide bodyParse limit

6107d73

bengourley merged commit 6107d73 into master

Contributor

bengourley commented Mar 24, 2016

Thanks for this. Couple of nit picks which I fixed up myself…

I changed the test names so that they read better
I fixed some style inconsistencies that weren't picked up by the linter
I documented this option in the readme

See the commit: 4183c23

Also, I don't like the > in your commit message. Be careful about mindlessly following a convention!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment