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

Add packet-loss and Docker-notes #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

henrikhorluck
Copy link

@henrikhorluck henrikhorluck commented Apr 21, 2021

  • When disconnected from the network you should not be able to send packets
  • Docker notes are helpful if you want to test it yourself, since it is not obvious that you need the capability or that docker-compose is unsuitable

Note that the proposed command for turning on network-sending loss will crash current network_rust-users here:
https://github.com/TTK4145/network-rust/blob/711059088905305a5c70b523e85f3cfa885e2d7b/src/udpnet/peers.rs#L31

With this error: { code: 1, kind: PermissionDenied, message: "Operation not permitted" }, which is not that much different from the current issue when unplugging the ethernet-cable
Update: now resolved with release v0.2.0 of network-rust

- When disconnected from the network you should not be able to send packets
- Docker notes are helpful if you want to test it yourself, since it is not obvious that you need the capability or that docker-compose is unsuitable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant