Validator should throw error rather than return c.json({ success, message })

[kevbook]

What is the feature you are proposing?

Validator should throw error rather than return c.json({ success, message })
So we can use onError handler to format and send the right response to the client.

[yusukebe]

@kevbook

The onError() is not designed for the validation error. Write like the following:

app.post(
  '/',
  validator('json', (data, c) => {
    if (!data.title) {
      return c.json({
        success: false
      }, 400)
    }
    return {
      title: data.title,
    }
  }),
  (c) => {
    return c.json({
      success: true
    }, 201)
  }
)

[yusukebe]

I see, thanks. Reopened the issue.

[ahmetbicer]

+1 for this. It would be nice to catch validator errors from onError handler.

[yusukebe]

Okay. We have to consider making it. But, it may be a breaking change. So will be included in the next major version "v4". Thanks!

[jvhellemond]

+1
I like to keep all my error responses uniform, with (translated) messages form malformed JSON or FormData generated by onError.