Update requests implementation

Author: macieyng

instrumentation/opentelemetry-instrumentation-requests/src/opentelemetry/instrumentation/requests/__init__.py

@@ -245,8 +245,16 @@ def _uninstrument_from(instr_root, restore_as_bound_func=False):
 
 
 def get_default_span_name(method):
-    """Default implementation for name_callback, returns HTTP {method_name}."""
-    return f"HTTP {method.strip()}"
+    """
+    Default implementation for name_callback, returns HTTP {method_name}.
+    https://opentelemetry.io/docs/reference/specification/trace/semantic_conventions/http/#name
+    
+    Args:
+        method: string representing HTTP method
+    Returns:
+        span name
+    """
+    return method.strip()
 
 
 class RequestsInstrumentor(BaseInstrumentor):

instrumentation/opentelemetry-instrumentation-requests/tests/test_requests_integration.py

@@ -116,7 +116,7 @@ def test_basic(self):
         span = self.assert_span()
 
         self.assertIs(span.kind, trace.SpanKind.CLIENT)
-        self.assertEqual(span.name, "HTTP GET")
+        self.assertEqual(span.name, "GET")
 
         self.assertEqual(
             span.attributes,
@@ -191,7 +191,7 @@ def name_callback(method, url):
         self.assertEqual(result.text, "Hello!")
         span = self.assert_span()
 
-        self.assertEqual(span.name, "HTTP GET")
+        self.assertEqual(span.name, "GET")
 
     def test_not_foundbasic(self):
         url_404 = "http://httpbin.org/status/404"

instrumentation/opentelemetry-instrumentation-requests/tests/test_requests_ip_support.py

@@ -71,7 +71,7 @@ def assert_success_span(self, response: requests.Response):
 
         span = self.assert_span()
         self.assertIs(trace.SpanKind.CLIENT, span.kind)
-        self.assertEqual("HTTP GET", span.name)
+        self.assertEqual("GET", span.name)
 
         attributes = {
             "http.status_code": 200,