mirror of
https://github.com/filebrowser/filebrowser.git
synced 2026-07-29 18:50:33 +00:00
0b925cbca9
A TUS PATCH rejected before its body was read left the client still streaming a chunk. net/http gives up draining an unread request body after 256KiB and then closes the connection, so the status we replied with was lost and the browser saw a bare transport error instead. The client could not tell a conflict from a network fault, retried blindly, and the upload stalled after the first chunk — while curl, which reads the response as it writes, still reported the status and appeared to work. Drain what is left of a rejected chunk so the answer arrives on a connection that stays usable. The explicit r.Body.Close() had to go with it: it ran before the drain, and the server closes the request body itself once the handler returns. Pressing Upload awaited a recursive listing of the whole destination subtree before a single byte was sent, with no sign that anything was happening. On a large destination that reads as a dead button, and the walk kept running after the client gave up, then reported the failed write as a 500. A flat upload can only collide with a direct child of the destination, so walk recursively only for folder uploads, stop the walk once the request is done, and show the action as busy while the destination is checked. Along the way: a failed upload was credited as fully sent, so the progress bar reported 100% when nothing had been written; the progress timer was recreated without clearing the previous one; and the chunk size field threw on a value with no unit and raced its own debounce on submit, so it appeared to refuse every change. Fixes #6006 Refs #5987 |
||
|---|---|---|
| .. | ||
| auth.go | ||
| auth_test.go | ||
| commands.go | ||
| data.go | ||
| headers.go | ||
| http.go | ||
| preview.go | ||
| preview_enum.go | ||
| public.go | ||
| public_symlink_test.go | ||
| public_test.go | ||
| raw.go | ||
| raw_test.go | ||
| resource.go | ||
| resource_checksum_test.go | ||
| resource_recursive_test.go | ||
| resource_test.go | ||
| rules_path_test.go | ||
| search.go | ||
| settings.go | ||
| share.go | ||
| share_test.go | ||
| static.go | ||
| subtitle.go | ||
| subtitle_test.go | ||
| tus_handlers.go | ||
| tus_multichunk_test.go | ||
| tus_symlink_test.go | ||
| tus_upload_length_test.go | ||
| upload_cache_memory.go | ||
| upload_cache_memory_test.go | ||
| upload_cache_redis.go | ||
| users.go | ||
| utils.go | ||
| utils_test.go | ||