400 instead of 422 about binding HOT 8 CLOSED

Yeah, you're right, 422 is kind of weird. And I think it does mean something slightly different than what we think of as a "Bad Request" - and seeing as this is a general package, the more common error code should be returned.

I'll make this change eventually. Swamped with school right now, but will get around to it.

from binding.

👍 +1 for 400: Bad Request :)

from binding.

sigh I got the commit message wrong, but the code is right. 81bd158

from binding.

hi, I always use 422, is there a way to define the status code you would like? Happy to do a PR that adds func (e Errors) Handle(response http.ResponseWriter, statusCode int) bool {

from binding.

Sure, we can see what people think.

from binding.

I'd rather not do this. If you want to control the status code and error message, simply pass in an http.ResponseWriter implementation that does writes the status code you want.

from binding.

Do you mean write my own Handle method not pass my own ResponseWriter?

from binding.

No, I mean pass your own http.ResponseWriter to Errors.Handle; maybe it looks something like this:

type BindingErrorWriter struct {
  http.ResponseWriter
  code int
}

func (w *BindingErrorWriter) WriteHeader(code int) {
  if code > 0 {
    return
  }

  if code == 422 {
    code = 400
  }
  w.code = code
  return w.ResponseWriter.WriteHeader(code)
}
func (w *BindingErrorWriter) Write(p []byte) (int, error) {
  if w.code == 0 {
    w.WriteHeader(200)
  }
  return w.ResponseWriter.Write(p)
}

from binding.