Fix two broken links on the News page - #162
Open
vipulpandey21 wants to merge 1 commit into
Open
Conversation
Addresses two of the links listed in theupdateframework#65. - The Forbes citation pointed at forbes.com/sites/.../uptane-will-protect-your-connected-car-from-hackers. The `...` is a literal placeholder in the source, not link rot, so the URL was never valid. The real article is "Uptane Will Protect Your Connected Car From Hackers" by Lee Mathews, Forbes, 2017-01-19. Confirmed live, and cross-referenced on Uptane's own press page (uptane.org/learn-more/press), which lists the same URL. - The LinuxCon+ContainerCon EU 2016 slides link (schd.ws) now 200s to an unrelated page (tech.blab.com), not a 404 htmltest would catch. Replaced with a Wayback Machine snapshot from 2022-12-13, confirmed to still be the original 8-page PDF (content-type application/pdf, valid PDF header). Also drops the two now-unused `.htmltest.yml` ignore entries for these URLs, since both replacements resolve normally. The remaining links in theupdateframework#65 need re-checking from a network that isn't filtering some of the destination domains, so this leaves theupdateframework#65 open rather than closing it. Signed-off-by: Vipul Subhash Pandey <vipulpandey7917@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Trivial fix per the first-time contributor policy (broken link), so no separate issue. Addresses two of the links listed in #65.
Forbes citation. The URL was
forbes.com/sites/.../uptane-will-protect-your-connected-car-from-hackers— the...is a literal placeholder in the source markdown, not link rot, so this URLwas never valid. The actual article is "Uptane Will Protect Your Connected Car
From Hackers" by Lee Mathews, Forbes, 2017-01-19:
forbes.com/sites/leemathews/2017/01/19/uptane-will-protect-your-connected-car-from-hackers/.Confirmed live (200), and the same URL is listed on Uptane's own press page
(uptane.org/learn-more/press), which I take as independent confirmation this is
the right article.
LinuxCon+ContainerCon EU 2016 slides.
schd.ws/hosted_files/...nowredirects to an unrelated page (
tech.blab.com) with a 200, sohtmltestwouldn't flag it — the link succeeds, it just isn't the slides anymore.
Replaced with a Wayback Machine snapshot from 2022-12-13, which I confirmed is
still the original file (
content-type: application/pdf, valid PDF header, 8pages).
Also removes the two now-unused
.htmltest.ymlignore entries for these URLs,since both replacements resolve normally.
The remaining links in #65 need checking from a network that isn't filtering
some of the destination domains (I'm getting inconsistent results depending on
network), so I'm leaving #65 open rather than closing it.
Diff is two files, six lines.
prettier --checkpasses on both.