'Get' example returns executable. about martini

[codegangsta]

I haven't seen it before, but I will check it out. Might be a bug in the static handler

Sent from my iPhone

On Nov 16, 2013, at 9:36 AM, Brent Anderson [email protected] wrote:

package main

import "github.com/codegangsta/martini"

func main() {
m := martini.Classic()
m.Get("/", func() string {
return "Hello world!"
})
m.Run()
}
Building, running and then browsing :3000 initiates a download of the built executable (5.3MB). Changing the path to '/test' and browsing :3000/test will return "Hello world!". This is on OSX Mavericks and tested with Go 1.1 and Go 1.2.

Running the default net/http server example on :3000 work just fine.

This is pretty bizarre behavior. Have you seen it before?

—
Reply to this email directly or view it on GitHub.

[codegangsta]

Looking into this now

[codegangsta]

I am not able to reproduce this with the latest version of martini (af1b7cc). This does seem like very odd behavior, and very insecure. If you could give me a little more information (log output, sample program, test case) it would be greatly appreciated

[DisposaBoy]

is the binary named public by any chance. e.g. you put main.go in a directory called public and then called go build or used a tool that does to run it

[codegangsta]

Hah, yeah that would be the only thing I can think of at the moment

On Sat, Nov 16, 2013 at 10:55 AM, DisposaBoy [email protected]:

is the binary named public by any chance. e.g. you put main.go in a
directory called public and then called go build or used a tool that does
to run it

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/50#issuecomment-28633127
.

[brentmn]

@DisposaBoy, exactly! I had thought about this but only tested by installing and running the binary from bin rather than renaming the directory; not the right test. @codegangsta, I'll rename my folder for now. Not sure if there's much else you can do about it.

[codegangsta]

@brentmn Not to much to do except documentation. Closing this issue out

[brentmn]

I would consider removing the default static folder or allowing it to be overwritten when calling martini.Static(...)