Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: throw on stream reset #26

Merged
merged 1 commit into from
May 1, 2023

Conversation

maschad
Copy link
Contributor

@maschad maschad commented Mar 15, 2023

Following the conversation here it seems that certain errors where not being propagated as an ERR_STREAM_RESET

maschad added a commit to maschad/js-libp2p-yamux that referenced this pull request Apr 19, 2023
@maschad maschad marked this pull request as ready for review April 19, 2023 16:17
@maschad maschad requested review from a team and achingbrain as code owners April 19, 2023 16:17
@maschad maschad marked this pull request as draft April 19, 2023 21:31
@maschad
Copy link
Contributor Author

maschad commented Apr 19, 2023

@maschad maschad force-pushed the fix/update-resetController branch from 265df2c to 887b24d Compare April 29, 2023 20:57
@maschad maschad marked this pull request as ready for review April 29, 2023 20:58
@maschad maschad marked this pull request as draft April 29, 2023 21:08
@maschad maschad force-pushed the fix/update-resetController branch from d939e6e to 5a46abe Compare May 1, 2023 16:48
@maschad maschad marked this pull request as ready for review May 1, 2023 16:48
@maschad maschad requested a review from wemeetagain May 1, 2023 16:49
@wemeetagain wemeetagain changed the title fix: improved error handling for stream resets fix: throw on stream reset May 1, 2023
@wemeetagain wemeetagain merged commit 6fc5ebd into ChainSafe:master May 1, 2023
github-actions bot pushed a commit that referenced this pull request May 1, 2023
## [4.0.1](v4.0.0...v4.0.1) (2023-05-01)

### Bug Fixes

* updated reset for abort controller ([#26](#26)) ([6fc5ebd](6fc5ebd))
@github-actions
Copy link

github-actions bot commented May 1, 2023

🎉 This PR is included in version 4.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants