security: Use quoted values for origin, URL, and force quoting.

This commit is contained in:
jamesread 2022-04-05 12:19:24 +01:00
parent 76971603b4
commit ebd1c4e938
3 changed files with 8 additions and 3 deletions

View File

@ -22,6 +22,11 @@ var (
)
func init() {
log.SetFormatter(&log.TextFormatter{
ForceQuote: true,
DisableTimestamp: true,
})
log.WithFields(log.Fields{
"version": version,
"commit": commit,

View File

@ -13,7 +13,7 @@ import (
func AllowCors(h http.Handler) http.Handler {
return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
if origin := r.Header.Get("Origin"); origin != "" {
log.Debugf("Adding CORS header origin: %v", origin)
log.Debugf("Adding CORS header origin: %q", origin)
w.Header().Set("Access-Control-Allow-Origin", origin)
}

View File

@ -32,12 +32,12 @@ func StartSingleHTTPFrontend(cfg *config.Config) {
mux := http.NewServeMux()
mux.HandleFunc("/api/", func(w http.ResponseWriter, r *http.Request) {
log.Debugf("api req: %v", r.URL)
log.Debugf("api req: %q", r.URL)
apiProxy.ServeHTTP(w, r)
})
mux.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
log.Debugf("ui req: %v", r.URL)
log.Debugf("ui req: %q", r.URL)
webuiProxy.ServeHTTP(w, r)
})