tools: add timers functions to the list of restricted globals · nodejs/node@01618c6 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 632
• Pull requests 754
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 01618c6

Browse filesBrowse the repository at this point in the historyBrowse files

aduh95

authored and

bengl

committed

tools: add timers functions to the list of restricted globals

PR-URL: #42013 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com>

1 parent a87fa5b commit 01618c6
Copy full SHA for 01618c6

2 files changed

+17-4Lines changed: 17 additions & 4 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib
  • .eslintrc.yaml
  • timers
    • promises.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/.eslintrc.yaml‎

Copy file name to clipboardExpand all lines: lib/.eslintrc.yaml

+12Lines changed: 12 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -83,6 +83,12 @@ rules:
83	83	message: Use `const { atob } = require('buffer');` instead of the global.
84	84	- name: btoa
85	85	message: Use `const { btoa } = require('buffer');` instead of the global.
	86	+ - name: clearImmediate
	87	+ message: Use `const { clearImmediate } = require('timers');` instead of the global.
	88	+ - name: clearInterval
	89	+ message: Use `const { clearInterval } = require('timers');` instead of the global.
	90	+ - name: clearTimeout
	91	+ message: Use `const { clearTimeout } = require('timers');` instead of the global.
86	92	- name: crypto
87	93	message: Use `const { crypto } = require('internal/crypto/webcrypto');` instead of the global.
88	94	- name: Crypto
@@ -99,6 +105,12 @@ rules:
99	105	message: Use `const { performance } = require('perf_hooks');` instead of the global.
100	106	- name: queueMicrotask
101	107	message: Use `const { queueMicrotask } = require('internal/process/task_queues');` instead of the global.
	108	+ - name: setImmediate
	109	+ message: Use `const { setImmediate } = require('timers');` instead of the global.
	110	+ - name: setInterval
	111	+ message: Use `const { setInterval } = require('timers');` instead of the global.
	112	+ - name: setTimeout
	113	+ message: Use `const { setTimeout } = require('timers');` instead of the global.
102	114	- name: structuredClone
103	115	message: Use `const { structuredClone } = require('internal/structured_clone');` instead of the global.
104	116	- name: SubtleCrypto

Collapse file

‎lib/timers/promises.js‎

Copy file name to clipboardExpand all lines: lib/timers/promises.js

+5-4Lines changed: 5 additions & 4 deletions

Original file line number	Diff line number	Diff line change
@@ -14,6 +14,11 @@ const {
14	14	Immediate,
15	15	insert
16	16	} = require('internal/timers');
	17	+ const {
	18	+ clearImmediate,
	19	+ clearInterval,
	20	+ clearTimeout,
	21	+ } = require('timers');
17	22
18	23	const {
19	24	AbortError,
@@ -73,7 +78,6 @@ function setTimeout(after, value, options = {}) {
73	78	insert(timeout, timeout._idleTimeout);
74	79	if (signal) {
75	80	oncancel = FunctionPrototypeBind(cancelListenerHandler,
76		- // eslint-disable-next-line no-undef
77	81	timeout, clearTimeout, reject, signal);
78	82	signal.addEventListener('abort', oncancel);
79	83	}
@@ -117,7 +121,6 @@ function setImmediate(value, options = {}) {
117	121	if (!ref) immediate.unref();
118	122	if (signal) {
119	123	oncancel = FunctionPrototypeBind(cancelListenerHandler,
120		- // eslint-disable-next-line no-undef
121	124	immediate, clearImmediate, reject,
122	125	signal);
123	126	signal.addEventListener('abort', oncancel);
@@ -153,7 +156,6 @@ async function* setInterval(after, value, options = {}) {
153	156	insert(interval, interval._idleTimeout);
154	157	if (signal) {
155	158	onCancel = () => {
156		- // eslint-disable-next-line no-undef
157	159	clearInterval(interval);
158	160	if (callback) {
159	161	callback(
@@ -175,7 +177,6 @@ async function* setInterval(after, value, options = {}) {
175	177	}
176	178	throw new AbortError(undefined, { cause: signal?.reason });
177	179	} finally {
178		- // eslint-disable-next-line no-undef
179	180	clearInterval(interval);
180	181	signal?.removeEventListener('abort', onCancel);
181	182	}

0 commit comments

Comments

0 (0)