diff --git a/lib/internal/quic/core.js b/lib/internal/quic/core.js index 037718b896e35c..0730303a134bc2 100644 --- a/lib/internal/quic/core.js +++ b/lib/internal/quic/core.js @@ -517,7 +517,6 @@ function createSecureContext(options, init_cb) { const sc_options = validateCreateSecureContextOptions(options); const { groups, earlyData } = sc_options; const sc = _createSecureContext(sc_options); - // TODO(@jasnell): Determine if it's really necessary to pass in groups here. init_cb(sc.context, groups, earlyData); return sc; } @@ -2702,7 +2701,6 @@ class QuicStream extends Duplex { } [kInspect](depth, options) { - // TODO(@jasnell): Proper custom inspect implementation const direction = this.bidirectional ? 'bidirectional' : 'unidirectional'; const initiated = this.serverInitiated ? 'server' : 'client'; return customInspect(this, { diff --git a/src/quic/node_quic_session.cc b/src/quic/node_quic_session.cc index 5bffad8a4fa524..999dbd71fda689 100644 --- a/src/quic/node_quic_session.cc +++ b/src/quic/node_quic_session.cc @@ -3378,8 +3378,10 @@ int QuicSession::OnStreamReset( // sensitivity of PATH_CHALLENGE operations (an attacker // could use a compromised PATH_CHALLENGE to trick an endpoint // into redirecting traffic). -// TODO(@jasnell): In the future, we'll want to explore whether -// we want to handle the different cases of ngtcp2_rand_ctx +// +// The ngtcp2_rand_ctx tells us what the random data is used for. +// Currently, there is only one use. In the future, we'll want to +// explore whether we want to handle the different cases uses. int QuicSession::OnRand( ngtcp2_conn* conn, uint8_t* dest, @@ -3744,8 +3746,7 @@ void QuicSessionSilentClose(const FunctionCallbackInfo& args) { session->Close(QuicSessionListener::SESSION_CLOSE_FLAG_SILENT); } -// TODO(addaleax): This is a temporary solution for testing and should be -// removed later. +// This is used purely for testing. void QuicSessionRemoveFromSocket(const FunctionCallbackInfo& args) { QuicSession* session; ASSIGN_OR_RETURN_UNWRAP(&session, args.Holder());