[starfish] add http.request.method attribute to http spans (#1633)

* add http.request.method to http spans

* update changelog

Author: buenaflor

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Enhancements
 
+- Add http.request.method attribute to http spans data ([#1633](https://github.com/getsentry/sentry-dart/pull/1633))
 - Add db.system and db.name attributes to db spans data ([#1629](https://github.com/getsentry/sentry-dart/pull/1629))
 
 ### Features

dart/lib/src/http_client/tracing_client.dart

@@ -41,7 +41,7 @@ class TracingClient extends BaseClient {
       span = null;
     }
 
-    span?.setData('http.method', request.method);
+    span?.setData('http.request.method', request.method);
     urlDetails?.applyToSpan(span);
 
     StreamedResponse? response;

dart/test/http_client/tracing_client_test.dart

@@ -38,7 +38,7 @@ void main() {
       expect(span.status, SpanStatus.ok());
       expect(span.context.operation, 'http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');

dio/lib/src/sentry_transformer.dart

@@ -27,7 +27,7 @@ class SentryTransformer implements Transformer {
           description: description,
         );
 
-    span?.setData('http.method', options.method);
+    span?.setData('http.request.method', options.method);
     // ignore: invalid_use_of_internal_member
     span?.origin = SentryTraceOrigins.autoHttpDioTransformer;
 
@@ -65,7 +65,7 @@ class SentryTransformer implements Transformer {
           description: description,
         );
 
-    span?.setData('http.method', options.method);
+    span?.setData('http.request.method', options.method);
     // ignore: invalid_use_of_internal_member
     span?.origin = SentryTraceOrigins.autoHttpDioTransformer;
 

dio/lib/src/tracing_client_adapter.dart

@@ -46,7 +46,7 @@ class TracingClientAdapter implements HttpClientAdapter {
       span = null;
     }
 
-    span?.setData('http.method', options.method);
+    span?.setData('http.request.method', options.method);
     urlDetails?.applyToSpan(span);
 
     ResponseBody? response;

dio/test/sentry_transformer_test.dart

@@ -39,7 +39,7 @@ void main() {
       expect(span.status, SpanStatus.ok());
       expect(span.context.operation, 'serialize.http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');
@@ -67,7 +67,7 @@ void main() {
       expect(span.status, SpanStatus.internalError());
       expect(span.context.operation, 'serialize.http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');
@@ -96,7 +96,7 @@ void main() {
       expect(span.status, SpanStatus.ok());
       expect(span.context.operation, 'serialize.http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');
@@ -126,7 +126,7 @@ void main() {
       expect(span.status, SpanStatus.internalError());
       expect(span.context.operation, 'serialize.http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');

dio/test/tracing_client_adapter_test.dart

@@ -43,7 +43,7 @@ void main() {
       expect(span.status, SpanStatus.ok());
       expect(span.context.operation, 'http.client');
       expect(span.context.description, 'GET https://example.com');
-      expect(span.data['http.method'], 'GET');
+      expect(span.data['http.request.method'], 'GET');
       expect(span.data['url'], 'https://example.com');
       expect(span.data['http.query'], 'foo=bar');
       expect(span.data['http.fragment'], 'baz');