Improper Output Neutralization for Logs [VID:117:github.com/govwa/util/middleware/middleware.go:23]

[veracode-workflow-app]

govwa//util/middleware/middleware.go

Lines 18 to 28 in 06b43e3

	}
	func (self *Class) LoggingMiddleware(h httprouter.Handle) httprouter.Handle {
	return func(w http.ResponseWriter, r *http.Request, ps httprouter.Params) {
	start := time.Now()
	log.Printf("Request From %s", r.Header.Get("User-Agent"))
	log.Printf("Started %s %s", r.Method, r.URL.Path)
	h(w, r, ps)
	log.Printf("Completed %s in %v", r.URL.Path, time.Since(start))
	}
	}

Filename: github.com/govwa/util/middleware/middleware.go

Line: 23

CWE: 117 (Improper Output Neutralization for Logs)

This call to log::Printf() could result in a log forging attack. Writing untrusted data into a log file allows an attacker to forge log entries or inject malicious content into log files. Corrupted log files can be used to cover an attacker's tracks or as a delivery mechanism for an attack on a log viewing or processing utility. For example, if a web administrator uses a browser-based utility to review logs, a cross-site scripting attack might be possible. The second argument to Printf() contains tainted data. The tainted data originated from an earlier call to github.com.govwa.util.middleware.!FuncLit6392418c.!func. Avoid directly embedding user input in log files when possible. Sanitize untrusted data used to construct log entries by using a safe logging mechanism such as the OWASP ESAPI Logger, which will automatically remove unexpected carriage returns and line feeds and can be configured to use HTML entity encoding for non-alphanumeric data. Alternatively, some of the XSS escaping functions from the OWASP Java Encoder project will also sanitize CRLF sequences. Only create a custom blocklist when absolutely necessary. Always validate untrusted input to ensure that it conforms to the expected format, using centralized data validation routines when possible. References: CWE OWASP Supported Cleansers/nDon't know how to fix this? Don't know why this was reported?
Get Assistance from Veracode