1761252 - Parse anchor attribute of Link header in EarlyHintPreloader

Status: RESOLVED FIXED

(Core :: Networking: HTTP, task, P2)

Description

[Manuel Bucher [:manuel]]

The Link spec requires the anchor attribute to be parsed or the complete Link reference to be ignored if an anchor exist.

RFC 8288, Section 3.2:

Link application can choose to ignore links with an anchor parameter.
For example, the application in use might not allow the link context
to be assigned to a different resource. In such cases, the entire
link is to be ignored; link applications MUST NOT process the link
without applying the anchor.

So this bug is about parsing the anchor attribute. Follow up on Bug 1753730

Comment 1

[Manuel Bucher [:manuel]]

Attachment: Bug 1761252 - Parse anchor attribute in Link-header r=#necko

Depends on D142202

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b70697d24e75
Parse anchor attribute in Link-header r=necko-reviewers,dragana,kershaw

Comment 3

[Sandor Molnar[:smolnar]]

Backed out 5 changesets (bug 1761242, bug 1744822, bug 1761252) for causing browser-chrome failures in netwerk/test/browser/browser_103_assets.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/d8e2d67cb443903c2fd609832b1395597ef843c1

Push with failures

Failure log

INFO - Buffered messages finished
[task 2022-06-08T15:51:36.858Z] 15:51:36     INFO - TEST-UNEXPECTED-FAIL | netwerk/test/browser/browser_103_assets.js | test_103_asset_normal (fetch): Unexpected amount of requests made - {"hinted":0,"normal":0} deepEqual {"hinted":0,"normal":1} - JS frame :: chrome://mochitests/content/browser/netwerk/test/browser/browser_103_assets.js :: test_hint_asset :: line 32
[task 2022-06-08T15:51:36.858Z] 15:51:36     INFO - Stack trace:
[task 2022-06-08T15:51:36.858Z] 15:51:36     INFO - chrome://mochitests/content/browser/netwerk/test/browser/browser_103_assets.js:test_hint_asset:32
[task 2022-06-08T15:51:36.859Z] 15:51:36     INFO - Leaving test bound test_103_asset_fetch

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b08e5b8d757a
Parse anchor attribute in Link-header r=necko-reviewers,dragana,kershaw

Comment 5

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/b08e5b8d757a