-
Notifications
You must be signed in to change notification settings - Fork 984
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(swarm): implement ConnectionHandler::poll_close
for combinators
#4794
Conversation
Follow-up to libp2p#4076. This is especially relevant since `libp2p-swarm-derive` uses `SelectConnectionHandler`.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh nice catch!
Did this land in v0.52
? I thought we only shipped this together with the removal of the handler
field from the error?
ConnectionHandler::poll_close
for combinators
@mxinden I fixed the compile errors and simplified some of the implementations by relying on the now documented requirement for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requiring fuse like behavior sounds good to me.
Thank you for the additional commits.
Mind approving this pull request @thomaseizinger? |
I am probably mistaken. Let's ignore |
Description
Follow-up to #4076.
This is especially relevant since
libp2p-swarm-derive
usesSelectConnectionHandler
.Notes & open questions
@thomaseizinger once this is merged, should we backport it to v0.52?
Change checklist