1953851 - URLs to MDN HTTP reference docs need to be updated ('Show MDN Docs' tooltip)

Status: RESOLVED FIXED

(DevTools :: Netmonitor, task)

Description

[Brian Thomas Smith [bsmth]]

The MDN HTTP reference docs will have a new scheme soon. The following sections have a Reference/ prefix now:

• Headers
• Methods
• Statuses

So, the new scheme looks like:

- Web/HTTP/Headers/
+ Web/HTTP/Reference/Headers/

- Web/HTTP/Status/
+ Web/HTTP/Reference/Status/

- Web/HTTP/Methods/
+ Web/HTTP/Reference/Methods/

Redirects are in place on MDN for all existing URLs, but we should update the URL prefix used in netmonitor to avoid hitting 301s.

See https://searchfox.org/mozilla-release/rev/02a9c1b3286ae3b7ca322b47e55fddb83d4e371c/devtools/client/netmonitor/src/utils/doc-utils.js#126-134

For example;

function getHeadersURL(header) {
  const lowerCaseHeader = header.toLowerCase();
  const idx = SUPPORTED_HEADERS.findIndex(
    item => item.toLowerCase() === lowerCaseHeader
  );
  return idx > -1
    ? `${MDN_URL}Web/HTTP/Headers/${SUPPORTED_HEADERS[idx] + getGAParams()}`
    : null;
}

Guides are also updated, but I don't see any places in netmonitor affected by this.

Comment 1

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1953851 - [devtools] Update the MDN links r=#devtools

Comment 2

[Hubert Boma Manilla (:bomsy)]

Thanks for filing Brian,

Guides are also updated, but I don't see any places in netmonitor affected by this.

Just confirming we have links to guides errors , i see Web/HTTP/CORS/Errors/ has changed to Web/HTTP/Guides/CORS/Errors/ .. is this correct?

Comment 3

[Brian Thomas Smith [bsmth]]

Hi Hubert. Yes that's correct, thanks for changing it. I took a look at your patch and this makes sense.

For the record, this is the from -> to mapping for redirects for those docs.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9d11f76b4c4d
[devtools] Update the MDN links in devtools r=devtools-reviewers,nchevobbe

Comment 5

[Serban Stanca [:SerbanS]]

Backed out for causing devtools failures in browser_webconsole_cors_errors.js.

• Backout link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | devtools/client/webconsole/test/browser/browser_webconsole_cors_errors.js | Click on the link opens the expected page - Got "https://developer.mozilla.org/docs/Web/HTTP/Guides/CORS/ErrorsCORSDisabled?utm_source=devtools&utm_medium=firefox-cors-errors&utm_campaign=default", expected "https://developer.mozilla.org/docs/Web/HTTP/Guides/CORS/Errors/CORSDisabled?utm_source=devtools&utm_medium=firefox-cors-errors&utm_campaign=d

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f123a8f22b50
[devtools] Update the MDN links in devtools r=devtools-reviewers,nchevobbe

Comment 7

[Hubert Boma Manilla (:bomsy)]

(In reply to Brian Thomas Smith [bsmth] from comment #3)

Hi Hubert. Yes that's correct, thanks for changing it. I took a look at your patch and this makes sense.

For the record, this is the from -> to mapping for redirects for those docs.

Thanks brian!

Comment 8

[Serban Stanca [:SerbanS]]

Backed out for causing devtools failures in browser_webconsole_warning_group_cookies.js.

• Backout link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-FAIL | devtools/client/webconsole/test/browser/browser_webconsole_warning_group_cookies.js | Test timed out -

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/376442637e13
[devtools] Update the MDN links r=devtools-reviewers,nchevobbe,cookie-reviewers,valentin,fluent-reviewers,zeid,anti-tracking-reviewers,translations-reviewers,android-reviewers,flod,bvandersloot,jonalmeida,tthibaud

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/51582 for changes under testing/web-platform/tests

Comment 11

[Sandor Molnar[:smolnar]]

Backed out for causing build bustages

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

Push with failures

Failure log -> build bustage

Failure log -> lint rust vendor

Comment 12

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/48b3c1a705d4
[devtools] Update the MDN links r=devtools-reviewers,nchevobbe,cookie-reviewers,valentin,fluent-reviewers,shtrom,zeid,anti-tracking-reviewers,translations-reviewers,android-reviewers,flod,bvandersloot,jonalmeida,tthibaud

Comment 14

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/48b3c1a705d4

Comment 15

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot