Uh oh!
There was an error while loading. Please reload this page.
quic: add listEndpoints API - #63536
Conversation
nodejs-github-bot
commented
May 24, 2026
Review requested:
|
Signed-off-by: James M Snell <jasnell@gmail.com>
nodejs-github-bot
commented
May 24, 2026
jasnell
commented
May 24, 2026
@nodejs/quic @mcollina ... there is an API design question in this... right now, the module maintains a global set of The module-level |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@## main #63536 +/- ##
==========================================
+ Coverage 90.32% 90.37% +0.05%
==========================================
Files 730 730 Lines 234152 234521 +369 Branches 43900 44141 +241 ==========================================
+ Hits 211499 211954 +455 + Misses 14374 14328 -46 + Partials 8279 8239 -40
🚀 New features to boost your workflow:
|
martenrichter
commented
May 25, 2026
Please look at the discussion at: |
mcollina
commented
May 25, 2026
Overall yes, we should add an Agent system to make the reuse constrained and limited if someone needs that. |
jasnell
commented
May 25, 2026
If we're going to add an agent, then I think this PR needs to take a different approach. Moving to draft for now. |
Actually.. thinking about it more... a Specifically: When |
This comment was marked as outdated.
This comment was marked as outdated.
metcoder95
commented
May 29, 2026
+1 on the Agent mechanism |
Signed-off-by: James M Snell <jasnell@gmail.com> PR-URL: #63536 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
jasnell
commented
May 31, 2026
Landed in 0fe48b6 |
Signed-off-by: James M Snell <jasnell@gmail.com> PR-URL: #63536 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com> PR-URL: nodejs#63536 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com> PR-URL: nodejs#63536 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Adds the
listEndpointsAPI. Since we're maintaining a registry of endpoints to be re-used withconnect, this helps enumerate which endpoints are currently available.Also apply a few other minor cleanups
@nodejs/quic