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

Yaron's review #68

Merged
merged 2 commits into from
Jan 17, 2024
Merged

Yaron's review #68

merged 2 commits into from
Jan 17, 2024

Conversation

thomas-fossati
Copy link
Contributor

  • add fwd reference to the threat model when the problem is presented in the intro
  • RRC is a sub-protocol
  • separate description of attack from its mitigation
  • ack Yaron

Fix #67

* add fwd reference to the threat model when the problem is presented in the intro
* RRC is a sub-protocol
* separate description of attack from its mitigation
* ack Yaron

Fix #67

Signed-off-by: Thomas Fossati <thomas.fossati@linaro.org>
@@ -70,6 +70,11 @@ CID-address binding is updated in that endpoint's session state.
This is done in order to give the receiving endpoint confidence
that the sending peer is in fact reachable at the source address (and port) indicated in the received datagram.

{{regular}} of this document explains the fundamental mechanism that aims to reduce the DDoS attack surface.
Additionally, in {{enhanced}}, a more advanced address validation mechanism is discussed.
This mechanism is designed to counteract off-path attackers who try to place themselves on-path by racing packets that trigger address rebinding at the receiver.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just as proposal:

s/who try/trying/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done in c91df60

@hannestschofenig hannestschofenig merged commit 1b755c9 into master Jan 17, 2024
2 checks passed
@hannestschofenig hannestschofenig deleted the yaron-review branch January 17, 2024 12:33
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.

Yaron's review
3 participants