Wrong link leading to 404 error

[jbcormoreche]

There's a link (https://api.dart.dev/stable/dart-core/Map-class.html) in a comment of the file html_dart2js.dart that leads to a 404 error.

The correct link is: https://api.dart.dev/dart-core/Map-class.html

It can found at line 30136 here:
https://github.com/dart-lang/sdk/blob/main/sdk/lib/html/dart2js/html_dart2js.dart

[dart-github-bot]

Summary: Broken link in html_dart2js.dart (line 30136) points to a non-existent Dart API page; correct link omits "stable/".

[parlough]

@athomas This also broke existing links from the Dart website. Is it possible to add a wildcard redirect so api.dart.dev/stable/dart-core/Map-class.html is redirected to api.dart.dev/dart-core/Map-class.html?

It seems like api.dart.dev/stable to api.dart.dev works, just not if any paths follow stable/.

[athomas]

As a first step, I created https://dart-review.googlesource.com/c/sdk/+/395961 to update the URLs to use the canonical version.

Similarly, the Dart website should not use the /stable/ path component unless it's linking to a specific version (/stable/1.2.3).

I believe I can add redirects for api.dart.dev/stable/dart-** -> api.dart.dev/dart-** which probably fixes most of those cases.

[parlough]

Thank you!

Similarly, the Dart website should not use the /stable/ path component unless it's linking to a specific version (/stable/1.2.3).

+1 Marya completed this in dart-lang/site-www@8c650ca. We'll make sure to follow this from now on as well.

I believe can add redirects for api.dart.dev/stable/dart-** -> api.dart.dev/dart-** which probably fixes most of those cases.

That sounds great and would be appreciated, as I imagine there are at least a few other links without an exact version across the internet.

[athomas]

Load balancer has been updated with the new redirect. The bug will be closed when the CL cleaning up the SDK has landed.