crypto: remove deprecated crypto._toBuf

PR-URL: nodejs#25338 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]>
refack · Jan 10, 2019 · fea4525 · fea4525
1 parent 548e94c
commit fea4525
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 15 deletions.
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -2167,15 +2167,18 @@ release.
 ### DEP0114: crypto._toBuf()
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url: https://github.com/nodejs/node/pull/???
+    description: End-of-Life.
   - version: v11.0.0
     pr-url: https://github.com/nodejs/node/pull/22501
     description: Runtime deprecation.
 -->
 
-Type: Runtime
+Type: End-Of-Life
 
 The `crypto._toBuf()` function was not designed to be used by modules outside
-of Node.js core and will be removed in the future.
+of Node.js core and was removed.
 
 <a id="DEP0115"></a>
 ### DEP0115: crypto.prng(), crypto.pseudoRandomBytes(), crypto.rng()

diff --git a/lib/crypto.js b/lib/crypto.js
@@ -94,8 +94,7 @@ const {
   getHashes,
   setDefaultEncoding,
   setEngine,
-  timingSafeEqual,
-  toBuf
+  timingSafeEqual
 } = require('internal/crypto/util');
 const Certificate = require('internal/crypto/certificate');
 
@@ -216,10 +215,6 @@ function getFipsForced() {
 }
 
 Object.defineProperties(exports, {
-  _toBuf: {
-    enumerable: false,
-    value: deprecate(toBuf, 'crypto._toBuf is deprecated.', 'DEP0114')
-  },
   createCipher: {
     enumerable: false,
     value: deprecate(createCipher,

diff --git a/test/parallel/test-crypto.js b/test/parallel/test-crypto.js
@@ -27,8 +27,7 @@ if (!common.hasCrypto)
 
 common.expectWarning({
   DeprecationWarning: [
-    ['crypto.createCipher is deprecated.', 'DEP0106'],
-    ['crypto._toBuf is deprecated.', 'DEP0114']
+    ['crypto.createCipher is deprecated.', 'DEP0106']
   ]
 });
 
@@ -301,8 +300,3 @@ testEncoding({
 testEncoding({
   defaultEncoding: 'latin1'
 }, assertionHashLatin1);
-
-{
-  // Test that the exported _toBuf function is deprecated.
-  crypto._toBuf(Buffer.alloc(0));
-}