Re: [whatwg/fetch] Strengthen requirements on CORS-safelisted request-headers (#736)

wanderview commented on this pull request.

Seems ok to me, but I defer to @ckerschb @bzbarsky here.

> @@ -414,38 +414,92 @@ each other by 0x2C 0x20, in order.
 
 <hr>
 
-<p id=simple-header>A <dfn export>CORS-safelisted request-header</dfn> is a <a for=/>header</a>
-whose <a for=header>name</a> is a <a>byte-case-insensitive</a> match for one of
+<p id=simple-header>To determine whether a <a for=/>header</a> <var>header</var> is a

Did you intent to produce text that says "header header" with the first header linked?

> @@ -414,38 +414,92 @@ each other by 0x2C 0x20, in order.
 
 <hr>
 
-<p id=simple-header>A <dfn export>CORS-safelisted request-header</dfn> is a <a for=/>header</a>
-whose <a for=header>name</a> is a <a>byte-case-insensitive</a> match for one of
+<p id=simple-header>To determine whether a <a for=/>header</a> <var>header</var> is a

Oh, I guess this is just `<type> <var name>`.  Its just confusing to me when the type and variable name are identical.

-- 
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/736#pullrequestreview-124966040

Received on Thursday, 31 May 2018 20:47:56 UTC