chore: add log context to proxy extension requests (#21834)

argoproj · Feb 14, 2025 · 6daaac5 · 6daaac5
1 parent 65664ce
commit 6daaac5
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/server/extension/extension.go b/server/extension/extension.go
@@ -781,7 +781,14 @@ func (m *Manager) CallExtension() func(http.ResponseWriter, *http.Request) {
 		user := m.userGetter.GetUser(r.Context())
 		groups := m.userGetter.GetGroups(r.Context())
 		prepareRequest(r, m.namespace, extName, app, user, groups)
-		m.log.Debugf("proxing request for extension %q", extName)
+		m.log.WithFields(log.Fields{
+			HeaderArgoCDUsername:        user,
+			HeaderArgoCDGroups:          strings.Join(groups, ","),
+			HeaderArgoCDNamespace:       m.namespace,
+			HeaderArgoCDApplicationName: fmt.Sprintf("%s:%s", app.GetNamespace(), app.GetName()),
+			"extension":                 extName,
+			"path":                      r.URL.Path,
+		}).Info("sending proxy extension request")
 		// httpsnoop package is used to properly wrap the responseWriter
 		// and avoid optional intefaces issue:
 		// https://github.com/felixge/httpsnoop#why-this-package-exists