w3c / beacon

Beacon
https://w3c.github.io/beacon/
Other
46 stars 22 forks source link

Typo in Processing Model - fetch step #56

Closed JosephPecoraro closed 6 years ago

JosephPecoraro commented 6 years ago

Typo in Processing Model. The Fetch step is missing a <dd> for origin's value, which should be origin. https://w3c.github.io/beacon/#sec-processing-model

Spec markup:

<dl>
  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#concept-request-method">method</a></dt>
  <dd><code>POST</code></dd>

  <dt><a href="#request-url">url</a></dt>
  <dd><var>parsedUrl</var></dd>

  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#concept-request-header-list">header list</a></dt>
  <dd><var>headerList</var></dd>

  <dt><a href="#request-origin">origin</a></dt>

  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#keep-alive-flag">keep-alive flag</a></dt>
  <dd><code>true</code></dd>

  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#concept-request-body">body</a></dt>
  <dd><var>transmittedData</var></dd>

  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#concept-request-mode">mode</a></dt>
  <dd><var>corsMode</var></dd>

  <dt><a data-link-type="dfn" href="https://fetch.spec.whatwg.org/#concept-request-credentials-mode">credentials mode</a></dt>
  <dd><i>include</i></dd>
</dl>

Namely:

<dt><a href="#request-origin">origin</a></dt>

Should be:

<dt><a href="#request-origin">origin</a></dt>
<dd><i>origin</i></dd>

Note that origin is defined at step 2.

igrigorik commented 6 years ago

@JosephPecoraro thanks, great catch!