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.

Sign up for GitHub

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

Jump to bottom

PMTU discovery #243

Open
agrover opened this issue Oct 14, 2019 · 0 comments · May be fixed by #1903
Open

PMTU discovery #243

agrover opened this issue Oct 14, 2019 · 0 comments · May be fixed by #1903
Labels
p3 Backlog task-large Should take more than 3 days (break up?)
Projects
2H 2020 bug triage

Comments

@agrover
Copy link
Contributor

agrover commented Oct 14, 2019

https://quicwg.org/base-drafts/draft-ietf-quic-transport.html#path-maximum-transmission-unit-pmtu

An endpoint SHOULD use Datagram Packetization Layer PMTU Discovery
([DPLPMTUD]) or implement Path MTU Discovery (PMTUD) [RFC1191] [RFC8201] to
determine whether the path to a destination will support a desired message size without
fragmentation.

In the absence of these mechanisms, QUIC endpoints SHOULD NOT send IP packets larger than 1280 bytes. Assuming the minimum IP header size, this results in a QUIC maximum packet size of 1232 bytes for IPv6 and 1252 bytes for IPv4. A QUIC implementation MAY be more conservative in computing the QUIC maximum packet size to allow for unknown tunnel overheads or IP header options/extensions.
@agrover agrover added this to the Future milestone Oct 14, 2019
agrover pushed a commit to agrover/neqo that referenced this issue Oct 14, 2019
Make Connection::pmtu a method
106c15e 
It can vary based on ipv4 or ipv6, or if we ever implemenet PMTUD (mozilla#243).

fixes mozilla#199
@agrover agrover mentioned this issue Oct 14, 2019
Merged
agrover pushed a commit to agrover/neqo that referenced this issue Oct 14, 2019
Make Connection::pmtu a method
2e8e223 
It can vary based on ipv4 or ipv6, or if we ever implemenet PMTUD (mozilla#243).

fixes mozilla#199
agrover pushed a commit to agrover/neqo that referenced this issue Oct 15, 2019
Make Connection::pmtu a method
ee47d89 
It can vary based on ipv4 or ipv6, or if we ever implemenet PMTUD (mozilla#243).

fixes mozilla#199
agrover pushed a commit that referenced this issue Oct 15, 2019
Make Connection::pmtu a method
dc76c79 
It can vary based on ipv4 or ipv6, or if we ever implemenet PMTUD (#243).

fixes #199
@agrover agrover mentioned this issue Nov 11, 2019
Closed
@martinthomson martinthomson changed the title Support PMTU discovery PMTU discovery Nov 26, 2019
@agrover agrover added this to Needs triage in 2H 2020 bug triage Jan 29, 2020
@agrover agrover moved this from Needs triage to Low priority in 2H 2020 bug triage Jan 29, 2020
@TheBlueMatt TheBlueMatt mentioned this issue Mar 18, 2021
Merged
@ddragana ddragana removed this from the Future milestone Mar 25, 2021
@martinthomson martinthomson added the task-large Should take more than 3 days (break up?) label Nov 4, 2021
@ddragana ddragana added the p3 Backlog label Nov 24, 2021
@larseggert larseggert linked a pull request May 14, 2024 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
p3 Backlog task-large Should take more than 3 days (break up?)
Projects
2H 2020 bug triage
Low priority
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: DPLPMTUD larseggert/neqo
3 participants
@agrover @martinthomson @ddragana