mirrors/dendrite
1
0
Fork 0
mirror of https://github.com/matrix-org/dendrite.git synced 2025-12-07 06:53:09 -06:00
Code Issues Packages Projects Releases Wiki Activity

mediaapi/writers/download: Reduce complexity of copyToActiveAndPassive

Browse source
This commit is contained in:
Robert Swain 2017-05-18 10:17:11 +02:00
parent b80d5ab919
commit 5348b64edc
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/github.com/matrix-org/dendrite/mediaapi/writers/download.go
View file

@ -297,7 +297,10 @@ func copyToActiveAndPassive(r io.Reader, wActive io.Writer, wPassive io.Writer,
break
}
bytesResponded += int64(bytesTemp)
if copyError == nil || (copyError != errFileIsTooLarge && copyError != errWrite) {
if copyError == nil {
// Note: if we get here then copyError != errFileIsTooLarge && copyError != errWrite
// as if copyError == errResponse || copyError == errWrite then we would have broken
// out of the loop and there are no other cases
// if larger than maxFileSize then stop writing to disk and discard cached file
if bytesWritten+int64(len(buffer)) > int64(maxFileSize) {
copyError = errFileIsTooLarge