Skip to content

Commit

Permalink
Fix https probe after 8.9.1 revert
Browse files Browse the repository at this point in the history
As the previous change was reverted, and won't go into the 8.x release
line, we only need to do this for 9.0.0 and up.

Refs: nodejs/node#16660
  • Loading branch information
gibfahn authored and mattcolegate committed Jan 15, 2018
1 parent 8ad628e commit cfe92c4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions probes/https-outbound-probe.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ var util = require('util');
var semver = require('semver');

var methods;
// In Node.js < v8.9.0 'get' calls 'request' so we only instrument 'request'
if (semver.lt(process.version, '8.9.0')) {
// In Node.js < v9.0.0 'get' calls 'request' so we only instrument 'request'
if (semver.lt(process.version, '9.0.0')) {
methods = ['request'];
} else {
methods = ['request', 'get'];
Expand Down

0 comments on commit cfe92c4

Please sign in to comment.