From 3754e527a728e3c54cbd44be357fb4cc97f623d8 Mon Sep 17 00:00:00 2001 From: ferologics Date: Wed, 14 Apr 2021 11:28:08 +0200 Subject: [PATCH] Address review comment - universally check if temp file size exceeds max file size --- mediaapi/routing/upload.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/mediaapi/routing/upload.go b/mediaapi/routing/upload.go index dd79a52a6..a38b56e09 100644 --- a/mediaapi/routing/upload.go +++ b/mediaapi/routing/upload.go @@ -158,8 +158,8 @@ func (r *uploadRequest) doUpload( } } - // If Content-Length header is not set, check if temp file size exceeds max file size configuration - if r.MediaMetadata.FileSizeBytes < 1 && bytesWritten > types.FileSizeBytes(*cfg.MaxFileSizeBytes) { + // Check if temp file size exceeds max file size configuration + if bytesWritten > types.FileSizeBytes(*cfg.MaxFileSizeBytes) { fileutils.RemoveDir(tmpDir, r.Logger) // delete temp file return requestEntityTooLargeJSONResponse(*cfg.MaxFileSizeBytes) }