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

Comments received during WG adoption call - RFC 7959 #2

Closed
emanjon opened this issue May 11, 2022 · 8 comments
Closed

Comments received during WG adoption call - RFC 7959 #2

emanjon opened this issue May 11, 2022 · 8 comments
Assignees

Comments

@emanjon
Copy link
Member

emanjon commented May 11, 2022

Mohamed Boucadair
https://mailarchive.ietf.org/arch/msg/core/4czSZYmRgMVgRA2d8mVpGUBwD0w/

It would helpful to explicit in Section 2.1 that this is about 7959, not the new block (to-be-RFC9177). Assessing the case of the new-block would be useful as well.

@emanjon
Copy link
Member Author

emanjon commented Nov 9, 2022

How does the attacks in draft-ietf-core-attacks-on-coap and the mitigations (Request-tag and ETag processing) defined in RFC 9175 apply to the Q-Block1 and Q-Block2 Options defined in 9177?

@emanjon
Copy link
Member Author

emanjon commented Nov 9, 2022

@chrysn can you make an update describing how the attacks applies or do not applies to Q-Block1 and Q-Block2 Options defined in 9177?

@chrysn
Copy link
Member

chrysn commented Apr 17, 2023

I'll take that one (and see what is left of #3 after doing that); my current assumption (which I'll check) is that the attack applies likewise. Then, the new text would say "This is about 7959 blockwise transfer. It applies likewise to 9177" (if that is true).

@emanjon
Copy link
Member Author

emanjon commented Apr 21, 2023

@chrysn Achim made the general comment:

With a list of preconditions it would be easier to see,
if a system is affected by that attack or not.

Check if preconditions for the fragment attack is stated clearly enough.

@emanjon
Copy link
Member Author

emanjon commented Apr 26, 2023

@chrysn @gselander

Do you think we could fix and close #2 and #3 and submit next week?

@chrysn
Copy link
Member

chrysn commented May 15, 2023

Sorry it took longer; there is now #8.

Check if preconditions for the fragment attack is stated clearly enough.

The section "Attack difficulty" that lists preconditions was already added after Achim's 2022-02 comments, and I think suffices.

@chrysn
Copy link
Member

chrysn commented May 15, 2023

Having checked the original comment: I think that once #8 is merged, this issue can be closed.

@emanjon
Copy link
Member Author

emanjon commented May 20, 2023

Closing as #8 has been mergerd

@emanjon emanjon closed this as completed May 20, 2023
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

No branches or pull requests

3 participants