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

Feedback by Éric Vyncke #225

Closed
hannestschofenig opened this issue Apr 5, 2021 · 2 comments
Closed

Feedback by Éric Vyncke #225

hannestschofenig opened this issue Apr 5, 2021 · 2 comments

Comments

@hannestschofenig
Copy link
Collaborator

== COMMENTS ==

-- Section 3 --
s/TLS cannot be used directly in datagram environments/TLS cannot be used directly over a datagram transport/ ?

[Hannes] DTLS is also used over non-IP-based networks (e.g. LoRaWAN, NB IoT, Bluetooth Smart) and "datagram transport". What would be the right term to describe "environments" that include transport layer protocols like UDP as well as those non-IP-based networks?

Bullet 2) s/to enable reassembly in the correct order/to enable reordering/ ?

[Hannes] The complete sentence is:

"
The DTLS handshake includes message
sequence numbers to enable reassembly in the correct order in
case datagrams are lost or reordered.
"

I believe that saying sequence numbers enable reassembly in the correct order is correct.

-- Section 3.1 --
Should there be a hint to a maximum retry count ?

[Hannes] There will be a retry count in an implementation. Do we need to say that if we don't have a number to suggest? Not sure.

-- Section 3.3 --
I understand the motivation (and no need to reply), but, sigh... implementing frag/reassembly above the transport layer...

[Hannes] Seems to be common these days. Examples: CoAP, and QUIC

@ekr ekr added the pr-exists label Apr 7, 2021
@kaduk
Copy link
Contributor

kaduk commented Apr 12, 2021

I don't think I found the PR to match up with the "pr-exists" label; could I get a pointer?

@ekr
Copy link
Collaborator

ekr commented Apr 13, 2021

My bad. #240

chris-wood added a commit that referenced this issue Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants