Skip to content

Commit 19e712b

Browse files
committed
quic: resolve InitializeSecureContext TODO comment
Using a static label is sufficient. PR-URL: #34741 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 2405922 commit 19e712b

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

src/quic/node_quic_crypto.cc

-2
Original file line numberDiff line numberDiff line change
@@ -602,8 +602,6 @@ void InitializeSecureContext(
602602
BaseObjectPtr<crypto::SecureContext> sc,
603603
bool early_data,
604604
ngtcp2_crypto_side side) {
605-
// TODO(@jasnell): Using a static value for this at the moment but
606-
// we need to determine if a non-static or per-session value is better.
607605
constexpr static unsigned char session_id_ctx[] = "node.js quic server";
608606
switch (side) {
609607
case NGTCP2_CRYPTO_SIDE_SERVER:

0 commit comments

Comments
 (0)