Skip to content
New issue

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

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

Already on GitHub? Sign in 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
src/stream.ts Outdated Show resolved Hide resolved
@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 📦🚀

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

Successfully merging this pull request may close these issues.

2 participants