Skip to content
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.

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

[Vote for this] Syntax error #70

Open
explosivebit opened this issue Dec 1, 2016 · 2 comments
Open

[Vote for this] Syntax error #70

explosivebit opened this issue Dec 1, 2016 · 2 comments
Assignees
Labels
enhancement question Further information is requested

Comments

@explosivebit
Copy link

In my opinion, in the query string
-- $ Wget 127.0.0.1:8081/tnt --post-data = '{ "method": "get_user_by_id", "params", [1]}'
There is a bug that should be so. "Params": [1] Instead of a comma "params", [1]

@dedok dedok added the bug Something isn't working label Dec 1, 2016
@dedok dedok self-assigned this Dec 1, 2016
@dedok
Copy link
Contributor

dedok commented Dec 1, 2016

Thanks for reporting! This is interesting, it looks like a bug inside libyajl (https://lloyd.github.io/yajl/). I'll check this issue soon.

@dedok
Copy link
Contributor

dedok commented Dec 1, 2016

Oh, I got it! You mean feature request not a bug :) Yep, nginx returns 404 code if error occurred. I'm not sure that people would happy if I'll break this behavior (means 404), so this issue will be stay opened until this issue doesn't collect some +1.

@dedok dedok added enhancement question Further information is requested and removed bug Something isn't working labels Dec 1, 2016
@dedok dedok changed the title Syntax error [Vote for this] Syntax error Dec 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants