From 3e5ac85ce1ca0ced4e639655f11107454c0cd4ce Mon Sep 17 00:00:00 2001 From: Robert Swain Date: Fri, 19 May 2017 10:53:47 +0200 Subject: [PATCH] mediaapi/writers/upload: Clarify TODO comment --- src/github.com/matrix-org/dendrite/mediaapi/writers/upload.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/github.com/matrix-org/dendrite/mediaapi/writers/upload.go b/src/github.com/matrix-org/dendrite/mediaapi/writers/upload.go index 79e5707e9..943fbb659 100644 --- a/src/github.com/matrix-org/dendrite/mediaapi/writers/upload.go +++ b/src/github.com/matrix-org/dendrite/mediaapi/writers/upload.go @@ -226,7 +226,7 @@ func storeFileAndMetadata(tmpDir types.Path, absBasePath types.Path, mediaMetada // This endpoint involves uploading potentially significant amounts of data to the homeserver. // This implementation supports a configurable maximum file size limit in bytes. If a user tries to upload more than this, they will receive an error that their upload is too large. // Uploaded files are processed piece-wise to avoid DoS attacks which would starve the server of memory. -// TODO: Requests time out if they have not received any data within the configured timeout period. +// TODO: We should time out requests if they have not received any data within a configured timeout period. func Upload(req *http.Request, cfg *config.MediaAPI, db *storage.Database) util.JSONResponse { logger := util.GetLogger(req.Context())