Fix #2618

[tomaka]

Fix #2618

It turns out that when a handshake timeout or protocol error happens, we set the connection's state to ShutdownWaitingAck { was_api_reset: false }.

was_api_reset: false indicates that the shutdown wasn't initiated through the public API.

Unfortunately, other parts of the code interpret was_api_reset: false as meaning "was initiated through the coordinator", as it assumes that a shutdown is only ever initiated either through the public API or through the coordinator.

This omits the third way to start a shutdown: when a handshake timeout or protocol error happens.

[mergify]

Automatically approving tomaka's pull requests. This auto-approval will be removed once more maintainers are active.

[github-actions]

twiggy diff report

Difference in .wasm size before and after this pull request.

---

 Delta Bytes │ Item
─────────────┼──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
        +145 ┊ core::ptr::drop_in_place<smoldot::libp2p::collection::multi_stream::MultiStreamConnectionTaskInner<smoldot_light_wasm::Instant,usize>>::h532ca1a6e5541b2f
        -145 ┊ core::ptr::drop_in_place<smoldot::libp2p::collection::multi_stream::MultiStreamConnectionTaskInner<smoldot_light_wasm::Instant,usize>>::hf3ca70f950db27ac
        +116 ┊ core::ptr::drop_in_place<smoldot::libp2p::collection::single_stream::SingleStreamConnectionTaskInner<smoldot_light_wasm::Instant>>::h51b347099058c3a5
        -116 ┊ core::ptr::drop_in_place<smoldot::libp2p::collection::single_stream::SingleStreamConnectionTaskInner<smoldot_light_wasm::Instant>>::h646874b33410ba30
         +33 ┊ smoldot::libp2p::collection::single_stream::SingleStreamConnectionTask<TNow>::inject_coordinator_message::h5ab2a482efb3be3c
          +1 ┊ smoldot::libp2p::collection::single_stream::SingleStreamConnectionTask<TNow>::read_write::h44de02e833671f05
         +34 ┊ Σ [6 Total Rows]