language-servers/deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-09-27 12:49:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

fix(ext/node): validate fs.close callback function (#30679)

Browse source 
Towards #29972

Allows the
[parallel/test-fs-close-errors.js](https://github.com/nodejs/node/blob/v24.2.0/test/parallel/test-fs-close-errors.js)
test to pass, and also addresses the `prefer-primordials` lint rule
#24236
This commit is contained in:
Daniel Osvaldo Rahmanto 2025-09-10 21:29:22 +07:00 • committed by GitHub
parent c6adba1228
commit 9c871d0a57
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 15 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

21
ext/node/polyfills/_fs/_fs_close.ts
View file

@ -1,14 +1,21 @@
// Copyright 2018-2025 the Deno authors. MIT license.
// TODO(petamoriken): enable prefer-primordials for node polyfills
// deno-lint-ignore-file prefer-primordials
import type { CallbackWithError } from "ext:deno_node/_fs/_fs_common.ts";
import {
type CallbackWithError,
makeCallback,
} from "ext:deno_node/_fs/_fs_common.ts";
import { getValidatedFd } from "ext:deno_node/internal/fs/utils.mjs";
import { core } from "ext:core/mod.js";
import { core, primordials } from "ext:core/mod.js";
const {
Error,
ErrorPrototype,
ObjectPrototypeIsPrototypeOf,
} = primordials;
export function close(fd: number, callback: CallbackWithError) {
fd = getValidatedFd(fd);
callback = makeCallback(callback);
setTimeout(() => {
let error = null;
try {
@ -16,7 +23,9 @@ export function close(fd: number, callback: CallbackWithError) {
// implementation detail and may change.
core.close(fd);
} catch (err) {
error = err instanceof Error ? err : new Error("[non-error thrown]");
error = ObjectPrototypeIsPrototypeOf(ErrorPrototype, err)
? err as Error
: new Error("[non-error thrown]");
}
callback(error);
}, 0);