Commit 51bca7d8 by bergquist

tech(dataproxy): moves all parameters to Info call

parent 839eb970
...@@ -18,6 +18,10 @@ import ( ...@@ -18,6 +18,10 @@ import (
"github.com/grafana/grafana/pkg/util" "github.com/grafana/grafana/pkg/util"
) )
var (
auditLogger log.Logger = log.New("data-proxy-log")
)
func NewReverseProxy(ds *m.DataSource, proxyPath string, targetUrl *url.URL) *httputil.ReverseProxy { func NewReverseProxy(ds *m.DataSource, proxyPath string, targetUrl *url.URL) *httputil.ReverseProxy {
director := func(req *http.Request) { director := func(req *http.Request) {
req.URL.Scheme = targetUrl.Scheme req.URL.Scheme = targetUrl.Scheme
...@@ -133,7 +137,6 @@ func ProxyDataSourceRequest(c *middleware.Context) { ...@@ -133,7 +137,6 @@ func ProxyDataSourceRequest(c *middleware.Context) {
func proxyLog(dataSourceType string, c *middleware.Context) { func proxyLog(dataSourceType string, c *middleware.Context) {
if setting.DataProxyLogging { if setting.DataProxyLogging {
auditLogger := log.New("data-proxy-log", "userid", c.UserId, "orgid", c.OrgId, "username", c.Login)
var body string var body string
if c.Req.Request.Body != nil { if c.Req.Request.Body != nil {
...@@ -142,6 +145,13 @@ func proxyLog(dataSourceType string, c *middleware.Context) { ...@@ -142,6 +145,13 @@ func proxyLog(dataSourceType string, c *middleware.Context) {
body = string(buffer) body = string(buffer)
} }
auditLogger.Info("Proxying incoming request", "datasource", dataSourceType, "uri", c.Req.RequestURI, "method", c.Req.Request.Method, "body", body) auditLogger.Info("Proxying incoming request",
"userid", c.UserId,
"orgid", c.OrgId,
"username", c.Login,
"datasource", dataSourceType,
"uri", c.Req.RequestURI,
"method", c.Req.Request.Method,
"body", body)
} }
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment