feat Modify the mapping of HTTP codes to OTel codes.

Author: xuerbujia

pkg/inst-api-semconv/instrumenter/http/http_status_code_extractor.go

@@ -34,6 +34,8 @@ func (h HttpClientSpanStatusExtractor[REQUEST, RESPONSE]) Extract(span trace.Spa
 		} else {
 			span.SetStatus(codes.Error, invalidHttpStatusCode)
 		}
+	} else if statusCode >= 200 && statusCode < 300 {
+		span.SetStatus(codes.Ok, "success")
 	}
 }
 
@@ -50,5 +52,7 @@ func (h HttpServerSpanStatusExtractor[REQUEST, RESPONSE]) Extract(span trace.Spa
 		} else {
 			span.SetStatus(codes.Error, invalidHttpStatusCode)
 		}
+	} else if statusCode >= 200 && statusCode < 300 {
+		span.SetStatus(codes.Ok, "success")
 	}
 }

pkg/inst-api-semconv/instrumenter/http/http_status_code_extractor_test.go

@@ -108,11 +108,23 @@ func TestHttpClientSpanStatusExtractor200(t *testing.T) {
 	u := codes.Code(0)
 	span := testSpan{status: &u}
 	c.Extract(span, nil, nil, nil)
-	if *span.status != codes.Unset {
-		panic("span status should be unset!")
+	if *span.status != codes.Ok {
+		panic("span status should be ok!")
+	}
+}
+func TestHttpClientSpanStatusExtractor201(t *testing.T) {
+	c := HttpClientSpanStatusExtractor[any, any]{
+		Getter: customizedNetHttpAttrsGetter{
+			code: 201,
+		},
+	}
+	u := codes.Code(0)
+	span := testSpan{status: &u}
+	c.Extract(span, nil, nil, nil)
+	if *span.status != codes.Ok {
+		panic("span status should be ok!")
 	}
 }
-
 func TestHttpServerSpanStatusExtractor500(t *testing.T) {
 	c := HttpServerSpanStatusExtractor[any, any]{
 		Getter: customizedNetHttpAttrsGetter{
@@ -150,7 +162,20 @@ func TestHttpServerSpanStatusExtractor200(t *testing.T) {
 	u := codes.Code(0)
 	span := testSpan{status: &u}
 	c.Extract(span, nil, nil, nil)
-	if *span.status != codes.Unset {
-		panic("span status should be unset!")
+	if *span.status != codes.Ok {
+		panic("span status should be ok!")
+	}
+}
+func TestHttpServerSpanStatusExtractor201(t *testing.T) {
+	c := HttpClientSpanStatusExtractor[any, any]{
+		Getter: customizedNetHttpAttrsGetter{
+			code: 201,
+		},
+	}
+	u := codes.Code(0)
+	span := testSpan{status: &u}
+	c.Extract(span, nil, nil, nil)
+	if *span.status != codes.Ok {
+		panic("span status should be ok!")
 	}
 }