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

Regression test "misbehaving" client #3949

Closed
Ben-PH opened this issue May 15, 2023 · 1 comment
Closed

Regression test "misbehaving" client #3949

Ben-PH opened this issue May 15, 2023 · 1 comment
Assignees
Labels
bootstrap Issues related to the bootstrap

Comments

@Ben-PH
Copy link
Contributor

Ben-PH commented May 15, 2023

#3946 was an issue due to code that looked like this in the bootstrap read-message section: https://github.com/massalabs/massa/blob/main/massa-bootstrap/src/client_binder.rs#L83-L86 and fixed with #3984

Regression test the system, e.g. by mocking a client that refuses to provide sufficient bytes, and ensure that the server drops the client.

@Ben-PH Ben-PH self-assigned this May 15, 2023
@Ben-PH Ben-PH added the bootstrap Issues related to the bootstrap label May 15, 2023
@Ben-PH
Copy link
Contributor Author

Ben-PH commented Jun 13, 2023

closing in favor of #4078

@Ben-PH Ben-PH closed this as completed Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bootstrap Issues related to the bootstrap
Projects
None yet
Development

No branches or pull requests

1 participant