From f44506588b954f897a9676c4560ba43ffd6ae7c1 Mon Sep 17 00:00:00 2001 From: Gibson Fahnestock Date: Mon, 15 Jan 2018 15:48:38 +0000 Subject: [PATCH] Fix https probe after 8.9.1 revert 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: https://github.com/nodejs/node/pull/16660 --- probes/https-outbound-probe.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/probes/https-outbound-probe.js b/probes/https-outbound-probe.js index 4b771961..81f0aa94 100644 --- a/probes/https-outbound-probe.js +++ b/probes/https-outbound-probe.js @@ -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'];