Commit 3745716b authored by Brad Fitzpatrick's avatar Brad Fitzpatrick

http: let FileServer work when path doesn't begin with a slash

... as when it's over-stripped with StripPrefix.

R=golang-dev, andybalholm, rsc
CC=golang-dev
https://golang.org/cl/4759052
parent caa56474
...@@ -242,7 +242,12 @@ func FileServer(root FileSystem) Handler { ...@@ -242,7 +242,12 @@ func FileServer(root FileSystem) Handler {
} }
func (f *fileHandler) ServeHTTP(w ResponseWriter, r *Request) { func (f *fileHandler) ServeHTTP(w ResponseWriter, r *Request) {
serveFile(w, r, f.root, path.Clean(r.URL.Path), true) upath := r.URL.Path
if !strings.HasPrefix(upath, "/") {
upath = "/" + upath
r.URL.Path = upath
}
serveFile(w, r, f.root, path.Clean(upath), true)
} }
// httpRange specifies the byte range to be sent to the client. // httpRange specifies the byte range to be sent to the client.
......
...@@ -10,6 +10,8 @@ import ( ...@@ -10,6 +10,8 @@ import (
"http/httptest" "http/httptest"
"io/ioutil" "io/ioutil"
"os" "os"
"path/filepath"
"strings"
"testing" "testing"
) )
...@@ -117,6 +119,36 @@ func TestFileServerCleans(t *testing.T) { ...@@ -117,6 +119,36 @@ func TestFileServerCleans(t *testing.T) {
} }
} }
func TestFileServerImplicitLeadingSlash(t *testing.T) {
tempDir, err := ioutil.TempDir("", "")
if err != nil {
t.Fatalf("TempDir: %v", err)
}
defer os.RemoveAll(tempDir)
if err := ioutil.WriteFile(filepath.Join(tempDir, "foo.txt"), []byte("Hello world"), 0644); err != nil {
t.Fatalf("WriteFile: %v", err)
}
ts := httptest.NewServer(StripPrefix("/bar/", FileServer(Dir(tempDir))))
defer ts.Close()
get := func(suffix string) string {
res, err := Get(ts.URL + suffix)
if err != nil {
t.Fatalf("Get %s: %v", suffix, err)
}
b, err := ioutil.ReadAll(res.Body)
if err != nil {
t.Fatalf("ReadAll %s: %v", suffix, err)
}
return string(b)
}
if s := get("/bar/"); !strings.Contains(s, ">foo.txt<") {
t.Logf("expected a directory listing with foo.txt, got %q", s)
}
if s := get("/bar/foo.txt"); s != "Hello world" {
t.Logf("expected %q, got %q", "Hello world", s)
}
}
func TestDirJoin(t *testing.T) { func TestDirJoin(t *testing.T) {
wfi, err := os.Stat("/etc/hosts") wfi, err := os.Stat("/etc/hosts")
if err != nil { if err != nil {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment