bbs.profullstack/agentbbs
1
0
Fork 0
mirror of https://github.com/profullstack/agentbbs.git synced 2026-08-13 14:27:27 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix(gopher): serve public files from SFTP storage (#115)

Browse source 
* test(gopher): cover SFTP public file paths

* fix(gopher): serve public files from SFTP storage
This commit is contained in:
RissRIce 2026-08-12 20:20:06 -06:00 • committed by GitHub
parent f83edd4351
commit be75744248
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 35 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

29
internal/gopher/gopher_test.go
View file

@ -159,6 +159,35 @@ func TestHomepageFileAndDir(t *testing.T) {
}
}
func TestPublicFilesUseSFTPStorageRoot(t *testing.T) {
srv, dataDir := newTestServer(t)
publicRoot := filepath.Join(dataDir, "files", "public", "alice")
mustMkdir(t, publicRoot)
mustWrite(t, filepath.Join(publicRoot, "shared.txt"), "uploaded through sftp")
legacyRoot := filepath.Join(dataDir, "users", "alice", "public")
mustMkdir(t, legacyRoot)
mustWrite(t, filepath.Join(legacyRoot, "legacy.txt"), "stale location")
dir := srv.Resolve("/files/~alice", false, "")
if dir.Kind != KindMenu {
t.Fatalf("public files root should be a dir menu, got %v", dir.Kind)
}
wire := string(dir.Wire())
if !strings.Contains(wire, "/files/~alice/shared.txt") {
t.Errorf("public files should list SFTP uploads:\n%s", wire)
}
if strings.Contains(wire, "legacy.txt") {
t.Errorf("public files must not read the legacy user directory:\n%s", wire)
}
file := srv.Resolve("/files/~alice/shared.txt", false, "")
if file.Kind != KindText || file.Text != "uploaded through sftp" {
t.Errorf("SFTP public file should be served as text: %+v", file)
}
}
func TestPathTraversalRefused(t *testing.T) {
srv, _ := newTestServer(t)
for _, sel := range []string{