ThreeMammals / Ocelot

.NET API Gateway
https://www.nuget.org/packages/Ocelot
MIT License
8.24k stars 1.62k forks source link

WebSocket Destination state abort Close fix #2091

Open hogwartsdeveloper opened 1 month ago

hogwartsdeveloper commented 1 month ago

Fixes #930

Proposed Changes

Screenshot 2024-06-09 at 02 29 46
raman-m commented 1 month ago

Zhannur, thank you for the pull request!

Firstly, have you managed to reproduce the issue through tests? I understand you've debugged the problem during runtime, but we need to verify the bug with tests. Therefore, unit and acceptance tests are necessary before the team can proceed with a review.

P.S.

Please, update your develop branch in forked repo, and pay attention I've rebased the feature branch, so you need to re-checkout the branch locally.

raman-m commented 1 month ago

@hogwartsdeveloper FYI