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

Change idle_timeout to max_idle_timeout #3099

Merged
merged 24 commits into from Jan 21, 2020
Merged

Change idle_timeout to max_idle_timeout #3099

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
61a3725
Idle timeout indicates you will timeout then
ianswett Oct 16, 2019
dc5b220
Update draft-ietf-quic-transport.md
ianswett Oct 16, 2019
2537a1a
Martin's suggestion and some reordering
ianswett Oct 29, 2019
61dcaad
Update draft-ietf-quic-transport.md
ianswett Oct 29, 2019
6fd0ac8
Update draft-ietf-quic-transport.md
ianswett Oct 29, 2019
e4dd819
Kazuho's suggestions
ianswett Oct 29, 2019
2234623
Update draft-ietf-quic-transport.md
ianswett Nov 1, 2019
11d8da8
Update draft-ietf-quic-transport.md
ianswett Nov 1, 2019
508413d
Update draft-ietf-quic-transport.md
ianswett Nov 1, 2019
e0d76ce
Update draft-ietf-quic-transport.md
ianswett Nov 1, 2019
48f04d6
Update draft-ietf-quic-transport.md
ianswett Nov 1, 2019
a7232f1
Update draft-ietf-quic-transport.md
ianswett Nov 2, 2019
ec80c86
Update draft-ietf-quic-transport.md
ianswett Nov 2, 2019
cd47d62
Update draft-ietf-quic-transport.md
ianswett Nov 2, 2019
7a6d513
Update draft-ietf-quic-transport.md
ianswett Nov 2, 2019
0bfcf11
Update draft-ietf-quic-transport.md
ianswett Nov 2, 2019
210821b
Merge branch 'master' into ianswett-symmetric-timeout
ianswett Nov 2, 2019
07d63d9
Jana's comment
ianswett Nov 3, 2019
e74856d
Update draft-ietf-quic-transport.md
ianswett Nov 18, 2019
581a368
Marten's suggestion
ianswett Nov 20, 2019
875de3f
Rebuild
ianswett Nov 20, 2019
774bc68
Use idle timer more directly
martinthomson Dec 11, 2019
c559540
wrap
martinthomson Dec 11, 2019
a203af6
Merge branch 'master' into ianswett-symmetric-timeout
martinthomson Jan 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion draft-ietf-quic-transport.md
View file
Open in desktop
Expand Up @@ -2349,7 +2349,7 @@ and the state is discarded when it remains idle for longer than the max
max_idle_timeout (see {{transport-parameter-definitions}}) and three times the
ianswett marked this conversation as resolved.
Show resolved Hide resolved
current Probe Timeout (PTO).

Each endpoint advertises a different max idle timeout, but the effective value
Each endpoint advertises a different max_idle_timeout, but the effective value
ianswett marked this conversation as resolved.
Show resolved Hide resolved
is determined by taking the minimum of the two advertised values. By announcing
ianswett marked this conversation as resolved.
Show resolved Hide resolved
a max idle timeout, endpoints commit to initiating an immediate close
ianswett marked this conversation as resolved.
Show resolved Hide resolved
({{immediate-close}}) if it abandons a connection prior to the effective value.
ianswett marked this conversation as resolved.
Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand this sentence... what does it mean for an endpoint to initiate an immediate close if it abandons the connection prior to the min? I thought that the endpoint was to initiate an immediate close at the min. Why are we talking about abandoning?

Expand Down