diff options
author | Colin Ihrig <cjihrig@gmail.com> | 2022-09-07 09:11:16 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-09-07 09:11:16 -0400 |
commit | 3b1204eb2d9c5cdf21bb92f7c8923869477f0969 (patch) | |
tree | ec8ccce739841defa84764253e172ee2d3db5ae1 /cli/tests/unit | |
parent | 027d4d433dce32a3b715184b54e7fe6403dedec2 (diff) |
fix(core): make errors more resistant to tampering (#15789)
This commit makes error objects more resistant to
prototype tampering.
This bug was found when updating the deno_std Node compatibility
layer to Node 18. The Node test 'parallel/test-assert-fail.js'
was breaking std's assertion library.
Refs: https://github.com/denoland/deno_std/pull/2585
Diffstat (limited to 'cli/tests/unit')
-rw-r--r-- | cli/tests/unit/error_test.ts | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/cli/tests/unit/error_test.ts b/cli/tests/unit/error_test.ts index 444b0445a..f3c9e20e6 100644 --- a/cli/tests/unit/error_test.ts +++ b/cli/tests/unit/error_test.ts @@ -1,5 +1,5 @@ // Copyright 2018-2022 the Deno authors. All rights reserved. MIT license. -import { assert } from "./test_util.ts"; +import { assert, assertThrows, fail } from "./test_util.ts"; Deno.test("Errors work", () => { assert(new Deno.errors.NotFound("msg") instanceof Error); @@ -22,3 +22,11 @@ Deno.test("Errors work", () => { assert(new Deno.errors.Busy("msg") instanceof Error); assert(new Deno.errors.NotSupported("msg") instanceof Error); }); + +Deno.test("Errors have some tamper resistance", () => { + // deno-lint-ignore no-explicit-any + (Object.prototype as any).get = () => {}; + assertThrows(() => fail("test error"), Error, "test error"); + // deno-lint-ignore no-explicit-any + delete (Object.prototype as any).get; +}); |