quic: resolve InitializeSecureContext TODO comment

jasnell · jasnell · commit 19e712b9b298 · 2020-08-17T11:31:20.000-07:00
Using a static label is sufficient. PR-URL: #34741 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com>
diff --git a/src/quic/node_quic_crypto.cc b/src/quic/node_quic_crypto.cc
@@ -602,8 +602,6 @@ void InitializeSecureContext(
     BaseObjectPtr<crypto::SecureContext> sc,
     bool early_data,
     ngtcp2_crypto_side side) {
-  // TODO(@jasnell): Using a static value for this at the moment but
-  // we need to determine if a non-static or per-session value is better.
   constexpr static unsigned char session_id_ctx[] = "node.js quic server";
   switch (side) {
     case NGTCP2_CRYPTO_SIDE_SERVER:
