Skip to content

Validate and update links (STF-557)#181

Merged
horgh merged 2 commits into
mainfrom
greg/stf-557
Jun 5, 2026
Merged

Validate and update links (STF-557)#181
horgh merged 2 commits into
mainfrom
greg/stf-557

Conversation

@oschwald

@oschwald oschwald commented Jun 4, 2026

Copy link
Copy Markdown
Member

Adds a lychee link-checker config (lychee.toml) and a
CI workflow (.github/workflows/links.yml) that validates links in Markdown and
src/**/*.c on push, PR, and weekly schedule. Then validates all existing links
and updates the ones that were stale or redirecting.

The lychee config mirrors the shared MaxMind setup: max_redirects = 0 (so
moved links surface as errors), accepts 500..=599, and excludes the changelog
(Changes.md), build/generated dirs (build/, .libs/, autom4te.cache/,
docs/public/), the vendored maxmind-db/ submodule, and the t/ test
harness. .lycheecache is gitignored.

Links updated (old -> new):

  • dev.maxmind.com/geoip/geolite2-free-geolocation-data?lang=en -> .../geolite2-free-geolocation-data/?lang=en (README.md)
  • dev.maxmind.com/geoip/geolocate-an-ip/databases?lang=en -> .../geolocate-an-ip/databases/?lang=en (README.md)
  • www.maxmind.com/en/geoip2-databases -> www.maxmind.com/en/geoip-databases (README.md)
  • www.maxmind.com/en/support -> support.maxmind.com/knowledge-base (README.md)
  • github.com/SpiderLabs/ModSecurity/ -> github.com/owasp-modsecurity/ModSecurity (README.md)
  • http://www.apache.org/licenses/LICENSE-2.0 -> https://www.apache.org/licenses/LICENSE-2.0 (src/mod_maxminddb.c license header)

Final lychee result: 11 OK, 0 errors, 1 excluded (a file:// self-reference
anchor).

Part of STF-557.

🤖 Generated with Claude Code

Comment thread .github/workflows/links.yml Fixed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a configuration file for the Lychee link checker (lychee.toml), adds .lycheecache to .gitignore, and updates several URLs in README.md and src/mod_maxminddb.c to use HTTPS or correct endpoints. There are no review comments, so I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@oschwald oschwald force-pushed the greg/stf-557 branch 2 times, most recently from f890450 to fa70ccb Compare June 4, 2026 20:36
oschwald and others added 2 commits June 4, 2026 22:01
Part of STF-557.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Validated all links with lychee and updated those that redirected
elsewhere to their canonical destination:

- dev.maxmind.com/geoip/geolite2-free-geolocation-data?lang=en ->
  .../geolite2-free-geolocation-data/?lang=en (canonical trailing slash)
- dev.maxmind.com/geoip/geolocate-an-ip/databases?lang=en ->
  .../geolocate-an-ip/databases/?lang=en (canonical trailing slash)
- www.maxmind.com/en/geoip2-databases -> www.maxmind.com/en/geoip-databases
- www.maxmind.com/en/support -> support.maxmind.com/knowledge-base
- github.com/SpiderLabs/ModSecurity/ -> github.com/owasp-modsecurity/ModSecurity
- http://www.apache.org/licenses/LICENSE-2.0 ->
  https://www.apache.org/licenses/LICENSE-2.0 (license header)

Part of STF-557.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@horgh horgh merged commit 68f1622 into main Jun 5, 2026
9 checks passed
@horgh horgh deleted the greg/stf-557 branch June 5, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants