]> source.dussan.org Git - gitea.git/commit
Fix CSV render error (#17406)
authorKN4CK3R <admin@oldschoolhack.me>
Sun, 24 Oct 2021 22:42:32 +0000 (00:42 +0200)
committerGitHub <noreply@github.com>
Sun, 24 Oct 2021 22:42:32 +0000 (00:42 +0200)
commit44f2c27d28eec4ba14b91b0729cbee831357b09f
tree9034fe81931413fca3e00eabb2d25f22a8f00bbf
parentf99d50fc9f8baf406f32a491b214f8a13617d086
Fix CSV render error (#17406)

closed #17378

Both errors from #17378 were caused by  #15175.

Problem 1 (error with added file):
`ToUTF8WithFallbackReader` creates a `MultiReader` from a `byte[2048]` and the remaining reader. `CreateReaderAndGuessDelimiter` tries to read 10000 bytes from this reader but only gets 2048 because that's the first reader in the `MultiReader`. Then the `if size < 1e4` thinks the input is at EOF and just returns that.

Problem 2 (error with changed file):
The blob reader gets defer closed. That was fine because the old version reads the whole file into memory. Now with the streaming version the close needs to defer after the method.
modules/csv/csv.go
routers/web/repo/compare.go