qugalet/gotosocial
1
0
Fork 0
mirror of https://github.com/superseriousbusiness/gotosocial.git synced 2024-11-25 13:16:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[bugfix] Correctly handle range > content-length (#2395)

Browse source
This commit is contained in:
Jadeiin 2023-11-29 10:25:52 +00:00 committed by GitHub
parent 2eb8b8eeb4
commit 2cc264584e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
internal/api/fileserver/servefile.go
View file

@ -206,10 +206,11 @@ func serveFileRange(rw http.ResponseWriter, r *http.Request, src io.Reader, rng
return return
} }
if end > size { if end >= size {
// According to the http spec if end >= size the server should return the rest of the file // According to the http spec if end >= size the server should return the rest of the file
// https://www.rfc-editor.org/rfc/rfc9110#section-14.1.2-6 // https://www.rfc-editor.org/rfc/rfc9110#section-14.1.2-6
end = size - 1 end = size - 1
endRng = strconv.FormatInt(end, 10)
} }
} else { } else {
// No end supplied, implying file end // No end supplied, implying file end