-
Notifications
You must be signed in to change notification settings - Fork 239
Open
Labels
8.8-candidateagent-nodejsMake available for APM Agents project planning.Make available for APM Agents project planning.buggood first issue
Description
The agent has handling for setting transaction.result and transaction.outcome for http and http2, but not for the https instrumentation.
Here are small scripts for each of http, https, http2 that create a server and includes a comment showing the transaction that it current generates:
- http: https://gist.github.com/trentm/7e09fcb523e0996782ba0536e619492a
- https: https://gist.github.com/trentm/c32dd0091b129ebb119fa88fbf6f3c4a
- http2: https://gist.github.com/trentm/d90b746d3e7d03fe490287471b677be8
Here is the latter part of diff -u playhttp.js playhttps.js that shows the difference in the transaction produced (note that I've replaced variable values like the transaction id with vars like "$id"):
...
@@ -32,7 +37,7 @@
"transaction": {
"name": "GET unknown route",
"type": "request",
- "result": "HTTP 2xx",
+ "result": "success",
"id": "$id",
"trace_id": "$trace_id",
"subtype": null,
@@ -49,15 +54,15 @@
"method": "GET",
"url": {
"raw": "/",
- "protocol": "http:",
+ "protocol": "https:",
"hostname": "localhost",
"port": "$port",
"pathname": "/",
- "full": "http://localhost:$port/"
+ "full": "https://localhost:$port/"
},
"socket": {
"remote_address": "::1",
- "encrypted": false
+ "encrypted": true
},
"headers": {
"host": "localhost:$port",
@@ -79,7 +84,7 @@
"span_count": {
"started": 0
},
- "outcome": "success",
+ "outcome": "unknown",
"sample_rate": 1
}
}Metadata
Metadata
Assignees
Labels
8.8-candidateagent-nodejsMake available for APM Agents project planning.Make available for APM Agents project planning.buggood first issue