http: move OutboundMessage.prototype.flush to EOL

API was deprecated long ago. Move to end of life and remove.
nodejs · Feb 3, 2020 · 485d53e · 485d53e
1 parent 99c8c6d
commit 485d53e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 43 deletions.
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -44,6 +44,9 @@ However, the deprecation identifier will not be modified.
 ### DEP0001: `http.OutgoingMessage.prototype.flush`
 <!-- YAML
 changes:
+  - version: REPLACEME
+    pr-url:https://github.com/nodejs/node/pull/31164
+    description: End-of-Life
   - version:
     - v4.8.6
     - v6.12.0
@@ -54,9 +57,9 @@ changes:
     description: Runtime deprecation.
 -->
 
-Type: Runtime
+Type: End-of-Life
 
-The `OutgoingMessage.prototype.flush()` method is deprecated. Use
+`OutgoingMessage.prototype.flush()` has been removed. Use
 `OutgoingMessage.prototype.flushHeaders()` instead.
 
 <a id="DEP0002"></a>

diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -893,10 +893,6 @@ OutgoingMessage.prototype.flushHeaders = function flushHeaders() {
   this._send('');
 };
 
-OutgoingMessage.prototype.flush = internalUtil.deprecate(function() {
-  this.flushHeaders();
-}, 'OutgoingMessage.flush is deprecated. Use flushHeaders instead.', 'DEP0001');
-
 OutgoingMessage.prototype.pipe = function pipe() {
   // OutgoingMessage should be write-only. Piping from it is disabled.
   this.emit('error', new ERR_STREAM_CANNOT_PIPE());

diff --git a/test/parallel/test-http-flush.js b/test/parallel/test-http-flush.js