Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit 1ef86fc

Browse files
authored
Merge pull request #34 from cpuguy83/fix_double_mux_attach
Fix double attach on single http serve mux.
2 parents d9f54a5 + 49763f5 commit 1ef86fc

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

internal/commands/root/http.go

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -96,16 +96,11 @@ func setupHTTPServer(ctx context.Context, p provider.Provider, cfg *apiServerCon
9696
StreamIdleTimeout: cfg.StreamIdleTimeout,
9797
StreamCreationTimeout: cfg.StreamCreationTimeout,
9898
}
99-
api.AttachPodRoutes(podRoutes, mux, true)
10099

101-
var summaryHandlerFunc api.PodStatsSummaryHandlerFunc
102100
if mp, ok := p.(provider.PodMetricsProvider); ok {
103-
summaryHandlerFunc = mp.GetStatsSummary
104-
}
105-
podMetricsRoutes := api.PodMetricsConfig{
106-
GetStatsSummary: summaryHandlerFunc,
101+
podRoutes.GetStatsSummary = mp.GetStatsSummary
107102
}
108-
api.AttachPodMetricsRoutes(podMetricsRoutes, mux)
103+
api.AttachPodRoutes(podRoutes, mux, true)
109104

110105
s := &http.Server{
111106
Handler: mux,

0 commit comments

Comments
 (0)