From f94eec0218419a639c6c0019d8e4a48a894f7aff Mon Sep 17 00:00:00 2001 From: Jeremiah Senkpiel Date: Wed, 20 Dec 2017 21:50:19 -0500 Subject: [PATCH] http: convert utcDate to use setTimeout A sort-of follow-up to https://github.com/nodejs/node/pull/17704, this removes the last internal use of enroll(). PR-URL: https://github.com/nodejs/node/pull/17800 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater Reviewed-By: Evan Lucas Reviewed-By: Minwoo Jung Reviewed-By: Luigi Pinca Reviewed-By: Anna Henningsen --- lib/internal/http.js | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/internal/http.js b/lib/internal/http.js index 71e32498f359a1..2b9c948aeefb30 100644 --- a/lib/internal/http.js +++ b/lib/internal/http.js @@ -1,20 +1,21 @@ 'use strict'; -const timers = require('timers'); +const { setUnrefTimeout } = require('internal/timers'); var dateCache; function utcDate() { if (!dateCache) { const d = new Date(); dateCache = d.toUTCString(); - timers.enroll(utcDate, 1000 - d.getMilliseconds()); - timers._unrefActive(utcDate); + + setUnrefTimeout(resetCache, 1000 - d.getMilliseconds()); } return dateCache; } -utcDate._onTimeout = function() { + +function resetCache() { dateCache = undefined; -}; +} function ondrain() { if (this._httpMessage) this._httpMessage.emit('drain');