Skip to content

Commit f04c575

Browse files
authored
Remove redundant URL (#90)
The Reporting API now lets us provide a custom URL to associate the report with. The default is to use the containing document's URL, which isn't relevant for a NEL report; for NEL reports we now use the URL of the original request as the URL of the report. This was previously stored in the NEL body's `uri` field; this is now redundant, and can be removed.
1 parent 1a617fd commit f04c575

File tree

1 file changed

+2
-14
lines changed

1 file changed

+2
-14
lines changed

index.html

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ <h2>Introduction</h2>
101101
<dt>data</dt>
102102
<dd><pre class="highlight">
103103
{
104-
"uri": "https://www.example.com/resource",
105104
"referrer": "https://referrer.com/",
106105
"sampling_fraction": 1.0,
107106
"server_ip": "123.122.121.120",
@@ -889,12 +888,6 @@ <h2>Generate a network error report</h2>
889888
following properties: [[ECMA-262]]
890889

891890
<dl>
892-
<dt><code>uri</code></dt>
893-
<dd>
894-
<var>request</var>'s URL, with any <a>fragment</a> component
895-
removed.
896-
</dd>
897-
898891
<dt><code>referrer</code></dt>
899892
<dd>
900893
<var>request</var>'s referrer, as determined by the <a>referrer
@@ -1007,6 +1000,8 @@ <h2>Generate a network error report</h2>
10071000
<dd><var>policy</var>'s <a>reporting group</a></dd>
10081001
<dt>settings</dt>
10091002
<dd><var>request</var>'s <a data-cite="!HTML#environment-settings-object">environment settings object</a></dd>
1003+
<dt>url</dt>
1004+
<dd><var>request</var>'s URL</dd>
10101005
</dl>
10111006
</li>
10121007
</ol>
@@ -1222,7 +1217,6 @@ <h2>Sample Network Error Reports</h2>
12221217
"type": "network-error",
12231218
"url": "https://www.example.com/",
12241219
"body": {
1225-
"uri": "https://www.example.com/",
12261220
"sampling_fraction": 0.5,
12271221
"referrer": "http://example.com/",
12281222
"server_ip": "123.122.121.120",
@@ -1253,7 +1247,6 @@ <h2>Sample Network Error Reports</h2>
12531247
"type": "network-error",
12541248
"url": "https://widget.com/thing.js",
12551249
"body": {
1256-
"uri": "https://widget.com/thing.js",
12571250
"sampling_fraction": 1.0,
12581251
"referrer": "https://www.example.com/",
12591252
"server_ip": "",
@@ -1311,7 +1304,6 @@ <h2>DNS misconfiguration</h2>
13111304
"type": "network-error",
13121305
"url": "https://new-subdomain.example.com/",
13131306
"body": {
1314-
"uri": "https://new-subdomain.example.com/",
13151307
"sampling_fraction": 1.0,
13161308
"server_ip": "",
13171309
"protocol": "http/1.1",
@@ -1380,7 +1372,6 @@ <h2>Origins with multiple IP addresses</h2>
13801372
"type": "network-error",
13811373
"url": "https://example.com/",
13821374
"body": {
1383-
"uri": "https://example.com/",
13841375
"sampling_fraction": 1.0,
13851376
"server_ip": "192.168.0.1",
13861377
"protocol": "http/1.1",
@@ -1410,7 +1401,6 @@ <h2>Origins with multiple IP addresses</h2>
14101401
"type": "network-error",
14111402
"url": "https://example.com/",
14121403
"body": {
1413-
"uri": "https://example.com/",
14141404
"sampling_fraction": 1.0,
14151405
"server_ip": "192.168.0.2",
14161406
"protocol": "http/1.1",
@@ -1444,7 +1434,6 @@ <h2>Origins with multiple IP addresses</h2>
14441434
"type": "network-error",
14451435
"url": "https://example.com/",
14461436
"body": {
1447-
"uri": "https://example.com/",
14481437
"sampling_fraction": 1.0,
14491438
"server_ip": "192.168.0.3",
14501439
"protocol": "http/1.1",
@@ -1475,7 +1464,6 @@ <h2>Origins with multiple IP addresses</h2>
14751464
"type": "network-error",
14761465
"url": "https://example.com/",
14771466
"body": {
1478-
"uri": "https://example.com/",
14791467
"sampling_fraction": 1.0,
14801468
"server_ip": "192.168.0.1",
14811469
"protocol": "http/1.1",

0 commit comments

Comments
 (0)