| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112 |
- package handler
- import (
- "os"
- "path/filepath"
- "testing"
- )
- func TestCleanPageImageRelativePath(t *testing.T) {
- tests := []struct {
- name string
- in string
- want string
- ok bool
- }{
- {name: "single filename", in: "logo.png", want: "logo.png", ok: true},
- {name: "nested path", in: "images/logo.png", want: filepath.Join("images", "logo.png"), ok: true},
- {name: "dot prefix", in: "./logo.png", want: "logo.png", ok: true},
- {name: "url escaped slash", in: "images%2Flogo.png", want: filepath.Join("images", "logo.png"), ok: true},
- {name: "parent traversal", in: "../secret.png", ok: false},
- {name: "encoded parent traversal", in: "%2e%2e/secret.png", ok: false},
- {name: "backslash traversal", in: `images\secret.png`, ok: false},
- {name: "absolute path", in: "/etc/passwd", ok: false},
- {name: "encoded absolute path", in: "%2fetc/passwd", ok: false},
- {name: "encoded nul byte", in: "logo.png%00", ok: false},
- {name: "invalid escape", in: "logo.png%zz", ok: false},
- {name: "empty path", in: "", ok: false},
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- got, ok := cleanPageImageRelativePath(tt.in)
- if ok != tt.ok {
- t.Fatalf("ok = %v, want %v", ok, tt.ok)
- }
- if got != tt.want {
- t.Fatalf("path = %q, want %q", got, tt.want)
- }
- })
- }
- }
- func TestResolvePageImagePath(t *testing.T) {
- root := t.TempDir()
- pagesDir := filepath.Join(root, "pages")
- base := filepath.Join(pagesDir, "guide")
- if err := os.MkdirAll(filepath.Join(base, "images"), 0755); err != nil {
- t.Fatalf("create images dir: %v", err)
- }
- if err := os.WriteFile(filepath.Join(base, "logo.png"), []byte("fake"), 0644); err != nil {
- t.Fatalf("create direct image: %v", err)
- }
- if err := os.WriteFile(filepath.Join(base, "images", "logo.png"), []byte("fake"), 0644); err != nil {
- t.Fatalf("create image: %v", err)
- }
- got, ok := resolvePageImagePath(pagesDir, base, "logo.png")
- if !ok {
- t.Fatal("expected direct image path to be accepted")
- }
- want := mustEvalSymlinks(t, filepath.Join(base, "logo.png"))
- if got != want {
- t.Fatalf("path = %q, want %q", got, want)
- }
- got, ok = resolvePageImagePath(pagesDir, base, "images/logo.png")
- if !ok {
- t.Fatal("expected nested image path to be accepted")
- }
- want = mustEvalSymlinks(t, filepath.Join(base, "images", "logo.png"))
- if got != want {
- t.Fatalf("path = %q, want %q", got, want)
- }
- if got, ok := resolvePageImagePath(pagesDir, base, "../guide.md"); ok {
- t.Fatalf("expected traversal to be rejected, got %q", got)
- }
- }
- func TestResolvePageImagePathRejectsSymlinkEscape(t *testing.T) {
- root := t.TempDir()
- pagesDir := filepath.Join(root, "pages")
- base := filepath.Join(pagesDir, "guide")
- outside := filepath.Join(root, "outside")
- if err := os.MkdirAll(base, 0755); err != nil {
- t.Fatalf("create page dir: %v", err)
- }
- if err := os.MkdirAll(outside, 0755); err != nil {
- t.Fatalf("create outside dir: %v", err)
- }
- if err := os.WriteFile(filepath.Join(outside, "secret.png"), []byte("secret"), 0644); err != nil {
- t.Fatalf("create outside file: %v", err)
- }
- if err := os.Symlink(outside, filepath.Join(base, "images")); err != nil {
- t.Skipf("symlink not supported: %v", err)
- }
- if got, ok := resolvePageImagePath(pagesDir, base, "images/secret.png"); ok {
- t.Fatalf("expected symlink escape to be rejected, got %q", got)
- }
- }
- func mustEvalSymlinks(t *testing.T, path string) string {
- t.Helper()
- realPath, err := filepath.EvalSymlinks(path)
- if err != nil {
- t.Fatalf("eval symlinks for %q: %v", path, err)
- }
- return realPath
- }
|