feat(http2): add http context info

Author: Qard

lib/instrumentation/modules/http2.js

@@ -159,11 +159,25 @@ module.exports = function (http2, agent, { enabled }) {
       ins.bindEmitter(req)
 
       var urlObj = parseUrl(headers[':path'])
+      var method = headers[':method'] || 'GET'
       var path = urlObj.pathname
-      span.name = headers[':method'] + ' ' + host + path
+      var url = host + path
+      span.name = method + ' ' + url
+
+      var statusCode
+      req.on('response', (headers) => {
+        statusCode = headers[':status']
+      })
 
       req.on('end', () => {
         agent.logger.debug('intercepted http2 client end event %o', { id })
+
+        span.setHttpContext({
+          method,
+          status_code: statusCode,
+          url
+        })
+
         span.end()
       })
 

test/instrumentation/modules/http2.js

@@ -282,7 +282,12 @@ isSecure.forEach(secure => {
       t.ok(span, 'root transaction should have span')
       t.equal(span.type, 'external')
       t.equal(span.subtype, 'http2')
-      t.equal(span.name, `undefined http${secure ? 's' : ''}://localhost:${port}/sub`)
+      t.equal(span.name, `GET http${secure ? 's' : ''}://localhost:${port}/sub`)
+      t.deepEqual(span.context.http, {
+        method: 'GET',
+        status_code: 200,
+        url: `http${secure ? 's' : ''}://localhost:${port}/sub`
+      })
 
       server.close()
       t.end()