Return 400 Bad Content on json, xml, geojson, rds parsing errors.

[meztez]

Plumber should return a 400 Bad Content when a parser can't parse content. Invalid json, xml, etc...

Likely fix with the generalized plumberError condition approach.

[mbuet2ner]

This would be a very welcomed feature 😊.

As of now, I use a somewhat hacky approach for JSON errors. Certainly not perfect but maybe this helps someone in the meantime...

error_handler <- function(req, res, err) {
  if (!inherits(err, "api_error") &&
      grepl("lexical error", err$message, fixed = TRUE)) {
    res$status <- 400
    body <- "Received malformed JSON."
    logger::log_error(
      'malformed_json "{err}"'
    )
  }
  else if (!inherits(err, "api_error")) {
    res$status <- 500
    body <- "Internal server error."
    logger::log_error(
      'internal_error "{err}"'
    )
  } else {
    res$status <- err$status
    body <- err$message
  }
  res$body <- list(message = body)
}