Fix #47: Handle dropping the WsManager without creating a new Runtime and blocking on tasks

hyperliquid-dex / hyperliquid-rust-sdk

MIT License

57 stars 47 forks source link

Open Frixoe opened 3 days ago

Frixoe commented 3 days ago

The previous implementation in the PR #51 was logging an output alerting that creating a new runtime within an existing runtime is not recommended.

This PR fixes that by allowing the thread to exit without blocking on them.

The implementation has been tested.

Frixoe commented 3 days ago

@lmlmt Do let me know if this is satisfactory. #51 did the same but with the extra warning.