Skip to content
This repository has been archived by the owner on Jun 17, 2021. It is now read-only.

Commit

Permalink
Fix issue that http plugin can't got the operation name
Browse files Browse the repository at this point in the history
  • Loading branch information
ascrutae committed Apr 22, 2019
1 parent 9b1591c commit c603957
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/nodejs-agent/lib/plugins/http/http.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ module.exports = function(httpModule, instrumentation, contextManager) {
function wrapRequest(original) {
return function(options, callback) {
let contextCarrier = new ContextCarrier();
let span = contextManager.createExitSpan(options.pathname, options.host + ":" + options.port, contextCarrier);
let span = contextManager.createExitSpan(options.path, options.host + ":" + options.port, contextCarrier);
contextCarrier.pushBy(function(key, value) {
if (!options.hasOwnProperty("headers")) {
options.headers = {};
Expand Down

1 comment on commit c603957

@Gorden-Wang
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ascrutae Does this commit was published to 1.1.2 ?

#bash npm i skyapm-nodejs@last -S

image

Please sign in to comment.