crypto: set DEFAULT_ENCODING property to non-enumerable · nodejs/node@0f63d84 (original) (raw)

Skip to content

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sign up

Commit 0f63d84

aduh95guybedford

authored and

committed

crypto: set DEFAULT_ENCODING property to non-enumerable

Since it is a deprecated API, a deprecation warning is printed when loading crypto module from ESM. Making it non enumerable remove the deprecation warning and make the API non-available to named imports. PR-URL: #23222 Fixes: #23203Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Guy Bedford guybedford@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Sakthipriyan Vairamani thechargingvolcano@gmail.com Reviewed-By: Refael Ackermann refack@gmail.com Reviewed-By: Tobias Nießen tniessen@tnie.de

File tree

1 file changed

lines changed

1 file changed

lines changed

Original file line number Diff line number Diff line change
@@ -230,7 +230,7 @@ Object.defineProperties(exports, {
230 230 fipsForced ? setFipsForced : setFipsCrypto
231 231 },
232 232 DEFAULT_ENCODING: {
233 -enumerable: true,
233 +enumerable: false,
234 234 configurable: true,
235 235 get: deprecate(getDefaultEncoding,
236 236 'crypto.DEFAULT_ENCODING is deprecated.', 'DEP0091'),