Date: Tue, 11 Jun 2019 05:22:43 -0700
From: Dmitri Tikhonov <notifications@github.com>
Reply-To: quicwg/base-drafts
 <reply+AFTOJK32BTNMJ7V3MW7NLJ53BTHZHEVBNHHBWCV3SI@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2776/review/248137155@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2776@github.com>
References: <quicwg/base-drafts/pull/2776@github.com>
Subject: Re: [quicwg/base-drafts] Explain initial settings a little more
 (#2776)
Mime-Version: 1.0
Content-Type: multipart/alternative;
 boundary="--==_mimepart_5cff9d1311de3_589d3ff47c2cd96c999429";
 charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/97lbrxIDGjxCKYvfO_nV5NE1B08>


----==_mimepart_5cff9d1311de3_589d3ff47c2cd96c999429
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

dtikhonov commented on this pull request.



> @@ -1340,7 +1340,9 @@ setting is the default value. When a 0-RTT QUIC connection is being used, the
 initial value of each server setting is the value used in the previous session.
 Clients MUST store the settings the server provided in the session being resumed
 and MUST comply with stored settings until the current server settings are
-received.
+received.  A client uses these initial values in a situation where it wants to

How about

> A client uses these initial values when it wants to send requests...

Or perhaps remove the "want" altogether and simplify it further to

> A client uses these initial values to send requests...

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/quicwg/base-drafts/pull/2776#discussion_r292425143
----==_mimepart_5cff9d1311de3_589d3ff47c2cd96c999429
Content-Type: text/html;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

<p><b>@dtikhonov</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/quicwg/base-drafts/pull/2776#discussion_r292425143">draft-ietf-quic-http.md</a>:</p>
<pre style='color:#555'>&gt; @@ -1340,7 +1340,9 @@ setting is the default value. When a 0-RTT QUIC connection is being used, the
 initial value of each server setting is the value used in the previous session.
 Clients MUST store the settings the server provided in the session being resumed
 and MUST comply with stored settings until the current server settings are
-received.
+received.  A client uses these initial values in a situation where it wants to
</pre>
<p>How about</p>
<blockquote>
<p>A client uses these initial values when it wants to send requests...</p>
</blockquote>
<p>Or perhaps remove the "want" altogether and simplify it further to</p>
<blockquote>
<p>A client uses these initial values to send requests...</p>
</blockquote>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/quicwg/base-drafts/pull/2776?email_source=notifications&amp;email_token=AFTOJK7ULA3PA2Y5W5F3HMTPZ6KJHA5CNFSM4HV2GMMKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3FELQY#discussion_r292425143">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AFTOJK5ADGFIFMDLDGFDU5LPZ6KJHANCNFSM4HV2GMMA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AFTOJKZNMFYKMWXA3T3V3H3PZ6KJHA5CNFSM4HV2GMMKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3FELQY.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/quicwg/base-drafts/pull/2776?email_source=notifications\u0026email_token=AFTOJK7ULA3PA2Y5W5F3HMTPZ6KJHA5CNFSM4HV2GMMKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3FELQY#discussion_r292425143",
"url": "https://github.com/quicwg/base-drafts/pull/2776?email_source=notifications\u0026email_token=AFTOJK7ULA3PA2Y5W5F3HMTPZ6KJHA5CNFSM4HV2GMMKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3FELQY#discussion_r292425143",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>
----==_mimepart_5cff9d1311de3_589d3ff47c2cd96c999429--