how to detect invalid JSON and response some helpful message? about bodyparser

[dead-horse]

1941812

[dead-horse]

https://github.com/cojs/co-body/blob/master/lib/json.js#L36

error is thrown here, you can add this in your custom error handler:

try {
   yield bodyParser({
      formLimit: '200kb',
      jsonLimit: '1mb',
    }).call(this,next)
} catch (err) {}

[eisneim]

stupid me!! just after i post the issue , i realized i can do it in that way! thanks anyway

[eisneim]

this method will also have a problem, it will caught not only the JSON parse error but all the nested middleware's error and router's error , which means, app.on('error') will become useless.

how to make this better?

[dead-horse]

maybe we can add options.onerror for koa-bodyparser to support custom error handler?

[eisneim]

it would be great ! my current solution is:
add global error handler and check error type

if((err instanceof SyntaxError) && err.body ){
    error = {
        msg:'JSON对象语法错误，请检查',
        detail: err.toString(),
    }
}

[dead-horse]

koa-bodyparser@1.6.0

[eisneim]

pretty efficient!