fix for issue : https://github.com/Totodore/socketioxide/issues/627

[Julian332]

Motivation

fix for issue : #627

Solution

because connect fn is stucked, so it executed after on_disconnect ,this insert a entry(sid , eio(closed) ) to namespace , so remove this entry if found eio is closed

[codspeed-hq]

CodSpeed Performance Report

Merging #628 will not alter performance

_{Comparing Julian332:fix_async_connect_timeout (8949983) with main (59d98af)}

Summary

✅ 87 untouched

[Totodore]

Hey, thanks for your PR, could you also add a regression test please?

I'll try to do review this week

[Julian332]

Hey, thanks for your PR, could you also add a regression test please?

I'll try to do review this week

thanks for your replay, but which kind of regression test , I found some test about this fn's behaviour, they all are passed .

[Totodore]

By regression test I mean a test that fails without your fix and passes thanks to your fix.

[Julian332]

By regression test I mean a test that fails without your fix and passes thanks to your fix.

hello,I have add that test

[Totodore]

Fixed and released in socketioxide v0.18.1. Sorry for the delay @Julian332.