From f0c717b0a1793971846e53e0fc1423e67a43197b Mon Sep 17 00:00:00 2001 From: Robert Swain Date: Wed, 31 May 2017 14:33:49 +0200 Subject: [PATCH] mediaapi/writers/download: Escalate corrupt db/file cases to errors --- .../dendrite/mediaapi/writers/download.go | 21 ++++++++----------- 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/src/github.com/matrix-org/dendrite/mediaapi/writers/download.go b/src/github.com/matrix-org/dendrite/mediaapi/writers/download.go index 1df28645..0df0c30c 100644 --- a/src/github.com/matrix-org/dendrite/mediaapi/writers/download.go +++ b/src/github.com/matrix-org/dendrite/mediaapi/writers/download.go @@ -148,30 +148,27 @@ func (r *downloadRequest) doDownload(w http.ResponseWriter, cfg *config.MediaAPI func (r *downloadRequest) respondFromLocalFile(w http.ResponseWriter, absBasePath types.Path) *util.JSONResponse { filePath, err := fileutils.GetPathFromBase64Hash(r.MediaMetadata.Base64Hash, absBasePath) if err != nil { - // FIXME: Remove erroneous file from database? - r.Logger.WithError(err).Warn("Failed to get file path from metadata") + r.Logger.WithError(err).Error("Failed to get file path from metadata") return &util.JSONResponse{ - Code: 404, - JSON: jsonerror.NotFound(fmt.Sprintf("File with media ID %q does not exist", r.MediaMetadata.MediaID)), + Code: 500, + JSON: jsonerror.InternalServerError(), } } file, err := os.Open(filePath) defer file.Close() if err != nil { - // FIXME: Remove erroneous file from database? - r.Logger.WithError(err).Warn("Failed to open file") + r.Logger.WithError(err).Error("Failed to open file") return &util.JSONResponse{ - Code: 404, - JSON: jsonerror.NotFound(fmt.Sprintf("File with media ID %q does not exist", r.MediaMetadata.MediaID)), + Code: 500, + JSON: jsonerror.InternalServerError(), } } stat, err := file.Stat() if err != nil { - // FIXME: Remove erroneous file from database? - r.Logger.WithError(err).Warn("Failed to stat file") + r.Logger.WithError(err).Error("Failed to stat file") return &util.JSONResponse{ - Code: 404, - JSON: jsonerror.NotFound(fmt.Sprintf("File with media ID %q does not exist", r.MediaMetadata.MediaID)), + Code: 500, + JSON: jsonerror.InternalServerError(), } }