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

Review by CA - Datagram v Connectionless #1026

Closed
gorryfair opened this issue May 19, 2022 · 3 comments
Closed

Review by CA - Datagram v Connectionless #1026

gorryfair opened this issue May 19, 2022 · 3 comments
Assignees

Comments

@gorryfair
Copy link
Contributor

  • "Connection establishment is only a local operation for a Datagram
    transport (e.g., UDP(-Lite))": I think the criterion is not Datagram
    transports but "transports without a handshake" or "connectionless
    protocols" as it is used later in the document.
@gorryfair gorryfair self-assigned this May 19, 2022
@gorryfair
Copy link
Contributor Author

I wrote, and I suggest we use "connectionless protocols" explicitly mentioning examples of UDP and UDP-Lite when used as a transport without a handshake.

@gorryfair gorryfair changed the title Review by CA - Datgram v Coinnectionless Review by CA - Datgram v Connectionless May 19, 2022
@gorryfair gorryfair changed the title Review by CA - Datgram v Connectionless Review by CA - Datagram v Connectionless May 19, 2022
@mwelzl
Copy link
Contributor

mwelzl commented May 20, 2022

To whoever plans to address this: I checked occurrences of the word "datagram" in the architecture and API drafts, and I found (only) one sentence that needs fixing on similar grounds, I think, in the API draft:

Guidance on setting this value for datagram transports is provided in [RFC8085].

in Section 8.1.4 "Timeout for keep alive packets". Surely the guidance regarding this matter in the UDP usage RFC is related to the fact that UDP(-Lite) is connectionless, not that it's datagram based.

@gorryfair
Copy link
Contributor Author

Starting #1034 to fix this.

gorryfair added a commit that referenced this issue May 26, 2022
replace DG by CL.

Changed one oddity: /as to/ to /about/
tfpauly added a commit that referenced this issue Jun 3, 2022
 PR to resolve Datagram v Connectionless #1026
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

2 participants