Adds a note about obfuscating query name/ID from the logs.

pull/1389/head
James Phillips 2015-11-11 22:54:55 -08:00
parent 6634cd6567
commit da20e6668b
1 changed files with 9 additions and 0 deletions

View File

@ -310,6 +310,15 @@ func (s *HTTPServer) wrap(handler func(resp http.ResponseWriter, req *http.Reque
}
}
// TODO (slackpad) We may want to consider redacting prepared
// query names/IDs here since they are proxies for tokens. But,
// knowing one only gives you read access to service listings
// which is pretty trivial, so it's probably not worth the code
// complexity and overhead of filtering them out. You can't
// recover the token it's a proxy for with just the query info;
// you'd need the actual token (or a management token) to read
// that back.
// Invoke the handler
start := time.Now()
defer func() {