 Matt Holloway
|
dd239d8443
|
Initial OSS logging adapter for http (#2008)
* initial logging stack for http
* add metrics adapter
* fix linter issues
* make log fields generic
* Update pkg/github/server_test.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Remove unused SlogMetrics adapter
The slog-based metrics adapter was never used — OSS always uses
NoopMetrics and the remote server has its own DataDog-backed adapter.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Update pkg/github/dependencies.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* fmt
* change to use slog
* address feedback
* rename noop adapter to noop sink
* Update pkg/http/server.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* [WIP] [WIP] Address feedback on OSS logging adapter for http implementation (#2264)
* Initial plan
* Fix BaseDeps.Logger and BaseDeps.Metrics to return safe defaults when Obsv is nil
Agent-Logs-Url: https://github.com/github/github-mcp-server/sessions/53221b0b-abb4-4138-a147-3ce9e13b379a
Co-authored-by: mattdholloway <918573+mattdholloway@users.noreply.github.com>
* Fix nil metrics in server.go by passing metrics.NewNoopMetrics() to NewExporters
Agent-Logs-Url: https://github.com/github/github-mcp-server/sessions/53221b0b-abb4-4138-a147-3ce9e13b379a
Co-authored-by: mattdholloway <918573+mattdholloway@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: mattdholloway <918573+mattdholloway@users.noreply.github.com>
Co-authored-by: Matt Holloway <mattdholloway@github.com>
* replace nil with stubs
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
|
2026-03-31 13:10:22 +01:00 |
|