Skip to content

fix: catch stream sink errors#25

Merged
wemeetagain merged 3 commits intomasterfrom
cayman/catch-stream-sink-error
Mar 1, 2023
Merged

fix: catch stream sink errors#25
wemeetagain merged 3 commits intomasterfrom
cayman/catch-stream-sink-error

Conversation

@wemeetagain
Copy link
Copy Markdown
Member

Resolves #23

The stream sink did not catch its own errors. This can lead to a situation where a thrown error can be unhandled.

@wemeetagain wemeetagain requested review from a team and achingbrain as code owners March 1, 2023 19:07
@wemeetagain wemeetagain merged commit 7c7fd07 into master Mar 1, 2023
@wemeetagain wemeetagain deleted the cayman/catch-stream-sink-error branch March 1, 2023 19:19
github-actions Bot pushed a commit that referenced this pull request Mar 1, 2023
## [3.0.7](v3.0.6...v3.0.7) (2023-03-01)

### Bug Fixes

* catch stream sink errors ([#25](#25)) ([7c7fd07](7c7fd07))
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2023

🎉 This PR is included in version 3.0.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

alanshaw pushed a commit to elastic-ipfs/bitswap-peer that referenced this pull request Sep 18, 2023
This PR re-enables yamux in E-IPFS. Some historical context:

1. E-IPFS release with yamux made all pods unstable. We reverted adding
yamux in #195 and
created issue ChainSafe/js-libp2p-yamux#23
2. In the meantime this issue was addressed
ChainSafe/js-libp2p-yamux#25

Please note that deploying this needs attention and we should monitor
and perform canary release.

Closes elastic-ipfs/elastic-ipfs#25

Co-authored-by: Alan Shaw <alan.shaw@protocol.ai>
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.

Yamux causing problems in production Elastic IPFS deployment

1 participant