humanmade / aws-xray

HM Platform AWS X-Ray Integration
23 stars 4 forks source link

Do not include http data on CLI #73

Closed roborourke closed 4 years ago

roborourke commented 4 years ago

Fixes #72

roborourke commented 4 years ago

My other thought is that perhaps this should add data from the CLI invocation? Could do that in a follow up. I'm thinking things like $argv and $argc, the command itself, anything extra we can infer from WP CLI commands specifically.

roborourke commented 4 years ago

XRay shouldn't be run in a non HTTP context so the issue this addresses can be handled upstream.