quic: remove no-longer relevant TODO statements

jasnell · jasnell · commit 06664298fa07 · 2020-07-27T13:33:12.000-07:00
PR-URL: #34498 Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/lib/internal/quic/core.js b/lib/internal/quic/core.js
@@ -1858,7 +1858,6 @@ class QuicSession extends EventEmitter {
 
     if (!this[kHandshakePost]()) {
       if (typeof state.handshakeCompletePromiseReject === 'function') {
-        // TODO(@jasnell): Proper error
         state.handshakeCompletePromiseReject(
           new ERR_OPERATION_FAILED('Handshake failed'));
       }
@@ -2005,7 +2004,6 @@ class QuicSession extends EventEmitter {
       state.closePromiseResolve();
 
     if (typeof state.handshakeCompletePromiseReject === 'function') {
-      // TODO(@jasnell): Proper error
       state.handshakeCompletePromiseReject(
         new ERR_OPERATION_FAILED('Handshake failed'));
     }

Original file line number	Diff line number	Diff line change
`@@ -1858,7 +1858,6 @@ class QuicSession extends EventEmitter {`
`1858`	`1858`
`1859`	`1859`	`if (!this[kHandshakePost]()) {`
`1860`	`1860`	`if (typeof state.handshakeCompletePromiseReject === 'function') {`
`1861`		`- // TODO(@jasnell): Proper error`
`1862`	`1861`	`state.handshakeCompletePromiseReject(`
`1863`	`1862`	`new ERR_OPERATION_FAILED('Handshake failed'));`
`1864`	`1863`	`}`
`@@ -2005,7 +2004,6 @@ class QuicSession extends EventEmitter {`
`2005`	`2004`	`state.closePromiseResolve();`
`2006`	`2005`
`2007`	`2006`	`if (typeof state.handshakeCompletePromiseReject === 'function') {`
`2008`		`- // TODO(@jasnell): Proper error`
`2009`	`2007`	`state.handshakeCompletePromiseReject(`
`2010`	`2008`	`new ERR_OPERATION_FAILED('Handshake failed'));`
`2011`	`2009`	`}`