Files
vikunja/pkg/files/files_test.go
kolaente 94f42bd6b2 fix(files): derive file size from reader at creation boundary
Authoritative size now comes from the reader instead of the caller's
claim in CreateWithMimeAndSession. The migration import path accepts
attacker-controlled metadata (GHSA-qh78-rvg3-cv54), so trusting
realsize for the limit check allowed oversized uploads to be accepted
and stored.

measureReaderSize leaves the reader seeked to 0 so the measured value
matches the bytes storage backends will actually write.
2026-04-09 16:22:56 +00:00

225 lines
5.9 KiB
Go

// Vikunja is a to-do list application to facilitate your life.
// Copyright 2018-present Vikunja and contributors. All rights reserved.
//
// This program is free software: you can redistribute it and/or modify
// it under the terms of the GNU Affero General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// This program is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU Affero General Public License for more details.
//
// You should have received a copy of the GNU Affero General Public License
// along with this program. If not, see <https://www.gnu.org/licenses/>.
package files
import (
"bytes"
"image"
"image/png"
"io"
"os"
"testing"
"code.vikunja.io/api/pkg/config"
"code.vikunja.io/api/pkg/db"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
type testauth struct {
id int64
}
func (a *testauth) GetID() int64 {
return a.id
}
func TestCreate(t *testing.T) {
t.Run("Normal", func(t *testing.T) {
initFixtures(t)
ta := &testauth{id: 1}
content := []byte("testfile")
createdFile, err := Create(bytes.NewReader(content), "testfile", uint64(len(content)), ta)
require.NoError(t, err)
// Check the file was created correctly
file := &File{ID: createdFile.ID}
err = file.LoadFileMetaByID()
require.NoError(t, err)
assert.Equal(t, int64(1), file.CreatedByID)
assert.Equal(t, "testfile", file.Name)
assert.Equal(t, uint64(len(content)), file.Size)
})
t.Run("Too Large", func(t *testing.T) {
initFixtures(t)
// initFixtures resets FilesMaxSize, so override afterwards.
oldMax := config.FilesMaxSize.GetString()
require.NoError(t, config.SetMaxFileSizeMBytesFromString("1MB"))
t.Cleanup(func() { _ = config.SetMaxFileSizeMBytesFromString(oldMax) })
ta := &testauth{id: 1}
_, err := Create(bytes.NewReader(make([]byte, 2*1024*1024)), "big.bin", 0, ta)
require.Error(t, err)
assert.True(t, IsErrFileIsTooLarge(err))
})
}
func TestCreate_HardenedAgainstLyingCaller(t *testing.T) {
// Regression: GHSA-qh78-rvg3-cv54. Caller's claimed size must not
// influence the limit check or the stored size.
// setLimit must run AFTER initFixtures since initFixtures resets it.
setLimit := func(t *testing.T, size string) {
t.Helper()
oldMax := config.FilesMaxSize.GetString()
require.NoError(t, config.SetMaxFileSizeMBytesFromString(size))
t.Cleanup(func() {
_ = config.SetMaxFileSizeMBytesFromString(oldMax)
})
}
ta := &testauth{id: 1}
t.Run("content over limit, claimed size under limit", func(t *testing.T) {
initFixtures(t)
setLimit(t, "1MB")
payload := make([]byte, 2*1024*1024)
_, err := Create(bytes.NewReader(payload), "sneaky.bin", 1, ta)
require.Error(t, err)
assert.True(t, IsErrFileIsTooLarge(err))
})
t.Run("content under limit, claimed size over limit", func(t *testing.T) {
initFixtures(t)
setLimit(t, "1MB")
payload := []byte("hi")
f, err := Create(bytes.NewReader(payload), "honest.txt", 99999999999, ta)
require.NoError(t, err)
assert.Equal(t, uint64(len(payload)), f.Size)
})
t.Run("content exactly at limit", func(t *testing.T) {
initFixtures(t)
setLimit(t, "1MB")
payload := make([]byte, 1*1024*1024)
f, err := Create(bytes.NewReader(payload), "edge.bin", 0, ta)
require.NoError(t, err)
assert.Equal(t, uint64(1*1024*1024), f.Size)
})
}
func TestCreateDetectsMimeType(t *testing.T) {
initFixtures(t)
ta := &testauth{id: 1}
// Minimal valid PNG (1x1 pixel)
pngData := createMinimalPNG(t)
f, err := Create(bytes.NewReader(pngData), "test.png", uint64(len(pngData)), ta)
require.NoError(t, err)
assert.Equal(t, "image/png", f.Mime)
}
func TestCreateDetectsMimeTypePlainText(t *testing.T) {
initFixtures(t)
ta := &testauth{id: 1}
textData := []byte("hello world this is plain text")
f, err := Create(bytes.NewReader(textData), "readme.txt", uint64(len(textData)), ta)
require.NoError(t, err)
assert.Equal(t, "text/plain; charset=utf-8", f.Mime)
}
func createMinimalPNG(t *testing.T) []byte {
t.Helper()
img := image.NewRGBA(image.Rect(0, 0, 1, 1))
buf := &bytes.Buffer{}
require.NoError(t, png.Encode(buf, img))
return buf.Bytes()
}
func TestFile_Delete(t *testing.T) {
t.Run("Normal", func(t *testing.T) {
s := db.NewSession()
defer s.Close()
initFixtures(t)
f := &File{ID: 1}
err := f.Delete(s)
require.NoError(t, err)
})
t.Run("Nonexisting", func(t *testing.T) {
s := db.NewSession()
defer s.Close()
initFixtures(t)
f := &File{ID: 9999}
err := f.Delete(s)
require.Error(t, err)
assert.True(t, IsErrFileDoesNotExist(err))
})
}
func TestFile_LoadFileByID(t *testing.T) {
t.Run("Normal", func(t *testing.T) {
initFixtures(t)
f := &File{ID: 1}
err := f.LoadFileByID()
require.NoError(t, err)
})
t.Run("Nonexisting", func(t *testing.T) {
initFixtures(t)
f := &File{ID: 9999}
err := f.LoadFileByID()
require.Error(t, err)
assert.True(t, os.IsNotExist(err))
})
}
func TestFileSave_UsesStorage(t *testing.T) {
originalStorage := storage
t.Cleanup(func() {
storage = originalStorage
})
mem := newMemStorage()
storage = mem
content := []byte("test-content")
file := &File{ID: 123, Size: uint64(len(content))}
err := file.Save(bytes.NewReader(content))
require.NoError(t, err)
rc, err := mem.Open(file.fileID())
require.NoError(t, err)
defer rc.Close()
written, err := io.ReadAll(rc)
require.NoError(t, err)
assert.Equal(t, content, written)
}
func TestFile_LoadFileMetaByID(t *testing.T) {
t.Run("Normal", func(t *testing.T) {
initFixtures(t)
f := &File{ID: 1}
err := f.LoadFileMetaByID()
require.NoError(t, err)
assert.Equal(t, "test", f.Name)
})
t.Run("Nonexisting", func(t *testing.T) {
initFixtures(t)
f := &File{ID: 9999}
err := f.LoadFileMetaByID()
require.Error(t, err)
assert.True(t, IsErrFileDoesNotExist(err))
})
}