-
Notifications
You must be signed in to change notification settings - Fork 0
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
CVPN-791: Add documentation about wireshark and PMTU #9
Conversation
Code coverage summary for 9570eee:
✅ Region coverage 60% passes |
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.
Re the wireguard bit, I wonder if maybe we should pull lightway-core/wireshark
up to the top level so it is a bit less hidden?
I'm also not sure it makes sense to have the plugin docs separate from the plugin code, perhaps it would be better to just leave them together and have a pointer from the docs
dir?
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.
Some really minor comments on the pmtu stuff.
612e558
to
9570eee
Compare
Add documentation about wireshark and PMTU