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
Don't force close connections if ping protocol is not supported by remote #2149
Conversation
28fe685
to
80ef250
Compare
80ef250
to
bc5a845
Compare
bc5a845
to
24cb5b3
Compare
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.
Couple of smaller comments and a merge conflict. Otherwise looks good to me.
@@ -189,6 +189,52 @@ fn max_failures() { | |||
QuickCheck::new().tests(10).quickcheck(prop as fn(_,_)) | |||
} | |||
|
|||
#[test] | |||
fn unsupported_doesnt_fail() { |
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.
🙏
`State` is explanatory enough and this is a private item.
Sorry for the failed build. Not yet used to my new RA-based setup. |
Still some CI failures which I can not tackle via GitHub suggestions. Can you take another look @thomaseizinger? |
@mxinden I had to switch the |
Making this configurable also allows us to delete some code from the relay tests.
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.
Thank you, also for the cleanup!
} | ||
|
||
#[derive(Clone)] | ||
pub struct KeepAliveBehaviour { |
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.
Thank you @thomaseizinger. Past-me should have done more research before writing this 😇
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.
I think a libp2p-test crate is desperately needed :D
Fixes #2109.