diff --git a/src/github.com/matrix-org/dendrite/clientapi/httputil/httputil.go b/src/github.com/matrix-org/dendrite/clientapi/httputil/httputil.go index d1f6c4d59..255ad1891 100644 --- a/src/github.com/matrix-org/dendrite/clientapi/httputil/httputil.go +++ b/src/github.com/matrix-org/dendrite/clientapi/httputil/httputil.go @@ -23,3 +23,11 @@ func UnmarshalJSONRequest(req *http.Request, iface interface{}) *util.JSONRespon } return nil } + +// LogThenError logs the given error then returns a matrix-compliant 500 internal server error response. +// This should be used to log fatal errors which require investigation. It should not be used +// to log client validation errors, etc. +func LogThenError(req *http.Request, err error) util.JSONResponse { + util.GetLogger(req.Context()).WithError(err).Error("request failed") + return jsonerror.InternalServerError() +} diff --git a/src/github.com/matrix-org/dendrite/clientapi/writers/createroom.go b/src/github.com/matrix-org/dendrite/clientapi/writers/createroom.go index 9f7f7efd9..77c112c28 100644 --- a/src/github.com/matrix-org/dendrite/clientapi/writers/createroom.go +++ b/src/github.com/matrix-org/dendrite/clientapi/writers/createroom.go @@ -168,11 +168,11 @@ func createRoom(req *http.Request, cfg config.ClientAPI, roomID string, producer } ev, err := buildEvent(&builder, builtEventMap, cfg) if err != nil { - return util.ErrorResponse(err) + return httputil.LogThenError(req, err) } if err := gomatrixserverlib.Allowed(*ev, &authEvents); err != nil { - return util.ErrorResponse(err) + return httputil.LogThenError(req, err) } // Add the event to the list of auth events @@ -183,10 +183,10 @@ func createRoom(req *http.Request, cfg config.ClientAPI, roomID string, producer // send events to the room server msgs, err := eventsToMessages(builtEvents, cfg.ClientAPIOutputTopic) if err != nil { - return util.ErrorResponse(err) + return httputil.LogThenError(req, err) } if err = producer.SendMessages(msgs); err != nil { - return util.ErrorResponse(err) + return httputil.LogThenError(req, err) } return util.JSONResponse{