Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upmaxParamLength Undocumented #1799
Open
Comments
|
@jkennedy1980 regards, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment


Bug Report
Restify Version
Documentation ( we're using an old version but the current doc. is still wrong )
Node.js Version
N/A
Expected behaviour
The documentation should explain the
maxParamLengthActual behaviour
The
maxParamLengthserver option should be included in the documentation. It's also a little strange that the server throws a 404 not found when a parameter is too long. It may be better to throw a 400 indicating the issue.Cause
That's the way it's written. Just need to update the documentation.
Are you willing and able to fix this?
Sure - where's the documentation? I can submit a code change in the coming weeks if a change is warranted.