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

CAD-3444 mux changes from p2p-master #3372

Merged
merged 4 commits into from
Sep 20, 2021
Merged

CAD-3444 mux changes from p2p-master #3372

merged 4 commits into from
Sep 20, 2021

Conversation

coot
Copy link
Contributor

@coot coot commented Sep 20, 2021

  • network-mux: label mini-protocol thread
  • network-mux: improve attenuated channel errors
  • network-mux: SDUSize show via Quiet
  • network-mux: mux runtime exceptions

@coot coot requested a review from karknu as a code owner September 20, 2021 13:50
@coot coot added networking mux issues related to network-mux labels Sep 20, 2021
@coot coot requested a review from dcoutts September 20, 2021 13:50
Copy link
Contributor

@karknu karknu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@coot
Copy link
Contributor Author

coot commented Sep 20, 2021

bors merge

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Sep 20, 2021

@iohk-bors iohk-bors bot merged commit 2ece89c into master Sep 20, 2021
@iohk-bors iohk-bors bot deleted the coot/p2p-mux branch September 20, 2021 22:39

data MuxRuntimeError =
ProtocolAlreadyRunning !MiniProtocolNum !MiniProtocolDir !MiniProtocolStatus
| UnknownProtocol !MiniProtocolNum !MiniProtocolDir
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UnknownProtocolInternalError

@coot coot added the peer2peer label Nov 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mux issues related to network-mux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants