fix: http.route may be empty in apollo otel spans which leads to an incomplete transaction name

Author: amychisholm03

lib/otel/attr-mapping/http.js

@@ -14,19 +14,19 @@ const attrMappings = {
   clientHost: {
     attrs: [constants.ATTR_SERVER_ADDRESS, constants.ATTR_NET_PEER_NAME],
     mapping() {
-      return () => {}
+      return () => { }
     }
   },
   clientPort: {
     attrs: [constants.ATTR_SERVER_PORT, constants.ATTR_NETWORK_PEER_PORT, constants.ATTR_NET_PEER_PORT],
     mapping() {
-      return () => {}
+      return () => { }
     }
   },
   clientUrl: {
     attrs: [constants.ATTR_FULL_URL, constants.ATTR_HTTP_URL],
     mapping() {
-      return () => {}
+      return () => { }
     }
   },
   grpcStatusCode: {
@@ -83,8 +83,16 @@ const attrMappings = {
     attrs: [constants.ATTR_HTTP_ROUTE],
     mapping({ segment, transaction }) {
       return (value) => {
-        transaction.nameState.appendPath(value)
-        segment.addAttribute('http.route', value)
+        if (value.length > 0) {
+          transaction.nameState.appendPath(value)
+          segment.addAttribute('http.route', value)
+        }
+        // Some spans e.g. Apollo Server do not have the http.route
+        // attribute and therefore will not have not a correct transaction
+        // name, so this is a work around.
+        else if (segment.attributes['http.target']) {
+          transaction.nameState.appendPath(segment.attributes['http.target'])
+        }
       }
     }
   },