Skip to content

Audit: concurrent-connection behavior in the Node adapter #30

Description

@smiggleworth

Purpose

Audit task, not a bug report — produces a test matrix, doesn't fix anything.

Procedure

Write or locate tests exercising genuinely concurrent connections/requests through the adapter:

Definition of Done

  • Table: scenario | ✅ test exists and passes / 🔗 gap filed.
  • No row left unmarked.

Audit results (2026-08-24)

Scenario Evidence Result
Concurrent real assets plus dotted and percent-encoded dynamic routes tests/node.test.tsshould let excluded dotted routes bypass static serving under concurrency
One connection aborts mid-request while other work continues should abort the Web Request when the Node request closes; shared real-socket conformance test
Slow/backpressured response alongside normal transport behavior should honor response backpressure; full concurrent suite
Shutdown races with an in-flight request should wait for an in-flight request before completing shutdown
Concurrent shutdown callers should close the application exactly once across concurrent shutdown

New coverage is included in #35; no concurrency defect remained.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions