Re: [whatwg/fetch] Initial resource timing integration. (#1185)

@annevk commented on this pull request.



> + <a for=/>unsafe shared current time</a> immmediately before starting the handshake process to
+ secure <var>connection</var>. [[!TLS]]
+
+ <li>
+  <p><var>timingInfo</var>'s <a for="connection timing info">alpn negotiated protocol</a> should be
+  the <var>connection</var>'s ALPN Protocol ID, with the following caveats: [[RFC7301]]
+  <!-- TODO Since it's in octets, I guess we have to isomorphic decode it? -->
+
+  <ul>
+   <li><p>When a proxy is configured, if a tunnel connection is established then this must be the
+   ALPN Protocol ID of the tunneled protocol, otherwise it must be the ALPN Protocol ID of the first
+   hop to the proxy.
+
+   <li><p>Octets in the ALPN Protocol ID must not be percent-encoded if they are valid token
+   characters except "%", and when using percent-encoding, uppercase hex digits must be used.
+   <!-- TODO I don't think we have to state this as the network does this. -->

That's only for HTTP headers though, how is that relevant here? In fact, I'm starting to think we should store this as a byte sequence and the API should use ByteString.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fetch/pull/1185#discussion_r600270094