contrib/valyala/fasthttp.v1: fix memory leak of spanOpts (#2962)

Co-authored-by: Pisit (King) Tangkoblarp <[email protected]>
DataDog · Nov 4, 2024 · 4d79271 · 4d79271
1 parent 2d679b9
commit 4d79271
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/contrib/valyala/fasthttp.v1/fasthttp.go b/contrib/valyala/fasthttp.v1/fasthttp.go
@@ -33,14 +33,14 @@ func WrapHandler(h fasthttp.RequestHandler, opts ...Option) fasthttp.RequestHand
 		fn(cfg)
 	}
 	log.Debug("contrib/valyala/fasthttp.v1: Configuring Middleware: cfg: %#v", cfg)
-	spanOpts := []tracer.StartSpanOption{
-		tracer.ServiceName(cfg.serviceName),
-	}
 	return func(fctx *fasthttp.RequestCtx) {
 		if cfg.ignoreRequest(fctx) {
 			h(fctx)
 			return
 		}
+		spanOpts := []tracer.StartSpanOption{
+			tracer.ServiceName(cfg.serviceName),
+		}
 		spanOpts = append(spanOpts, defaultSpanOptions(fctx)...)
 		fcc := &fasthttptrace.HTTPHeadersCarrier{
 			ReqHeader: &fctx.Request.Header,