openshift / geard

geard is no longer maintained - see OpenShift 3 and Kubernetes
Other
405 stars 80 forks source link

Remove extra HTTP headers from index.html in busybox-http-app #214

Closed mfojtik closed 10 years ago

wallrj commented 10 years ago

I think the text file will still be served with content-type HTML. Is it worth correcting that in this branch too? (see the last part of my description in #213)

mfojtik commented 10 years ago

@wallrj I think when I change that to index.txt then httpd will reply 404 for requests on / as the 'index.txt' is not set as Index file. But worth to try.

wallrj commented 10 years ago

Looks like you can configure it. See http://git.busybox.net/busybox/tree/networking/httpd.c#n615

And you can specify the location of the configuration file.

]$ docker run --tty --interactive openshift/busybox-http-app httpd --help
BusyBox v1.22.1 (2014-06-04 22:26:30 UTC) multi-call binary.

Usage: httpd [-ifv[v]] [-c CONFFILE] [-p [IP:]PORT] [-h HOME]
or httpd -d/-e STRING

Listen for incoming HTTP requests

    -i      Inetd mode
    -f      Don't daemonize
    -v[v]       Verbose
    -p [IP:]PORT    Bind to IP:PORT (default *:80)
    -h HOME     Home directory (default .)
    -c FILE     Configuration file (default {/etc,HOME}/httpd.conf)
    -e STRING   HTML encode STRING
    -d STRING   URL decode STRING
smarterclayton commented 10 years ago

[merge]

openshift-bot commented 10 years ago

Origin Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_geard/29/) (Image: devenv-fedora_94)

openshift-bot commented 10 years ago

Evaluated for origin up to 766b90847cb06278c06a5bcdac84a807388207b8