Skip to content

Stop one offline host from failing the link check - #1

Merged
sr-dash merged 1 commit into
mainfrom
fix/link-check-timeouts
Oct 10, 2026
Merged

sr-dash merged 1 commit into
mainfrom
fix/link-check-timeouts

Conversation

@sr-dash

@sr-dash sr-dash commented Oct 10, 2026

Copy link
Copy Markdown
Owner

CI has failed on every run since 5 October, always on the same single link: the Hinode XRT flare catalogue (https://xrt.cfa.harvard.edu/flare_catalog/) on the Resources page. 846 of 847 links passed, zero errors, one timeout.

The link is correct and still canonical. The host is offline: no answer on port 443 or 80, while www.cfa.harvard.edu responds normally.

Change (lychee.toml only):

  • accept_timeouts = true: a timeout no longer fails the build. 404s, DNS and TLS failures still do, and timeouts are still listed in the job summary.
  • Removes the gong.nso.edu, jsoc.stanford.edu and sidc.be excludes, which existed only because those hosts time out from runners. They're checked again.

This PR's own CI run is the test: it runs the link check on GitHub's runners with the new config.

Every CI run since 5 October failed on a single link: the Hinode XRT flare
catalogue on the Resources page. The URL is correct and still canonical —
the host, xrt.cfa.harvard.edu, stopped answering on both ports while the
rest of cfa.harvard.edu stayed up. 846 of 847 links passed with no errors;
the failure was one timeout.

A timeout is not evidence of a dead link, so lychee now accepts them. A 404,
a DNS failure or a TLS error still fails the build, and timeouts still show
in the job summary, so a host that never comes back stays visible.

That also retires an older workaround. gong.nso.edu, jsoc.stanford.edu and
sidc.be were excluded outright because they time out from GitHub's runners,
which left those URLs unchecked. They are back under the check.
@sr-dash
sr-dash merged commit 9a68914 into main Oct 10, 2026
2 checks passed
@sr-dash
sr-dash deleted the fix/link-check-timeouts branch October 10, 2026 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant