Skip to content

fix(en): replace deprecated Lucia deep-links with working targets - #14692

Closed
auroraxo wants to merge 1 commit into
withastro:mainfrom
auroraxo:fix-lucia-dead-links
Closed

auroraxo wants to merge 1 commit into
withastro:mainfrom
auroraxo:fix-lucia-dead-links

Conversation

@auroraxo

Copy link
Copy Markdown
Contributor

Description (required)

The Lucia section of the authentication guide links three subpages that no longer exist. I verified each one directly with curl (HTTP 404, October 2026):

Lucia itself was deprecated in March 2025 (https://pilcrowonpaper.com/blog/18), and the site now serves only a landing page pointing at a single-file replacement and the Auth Book.

Change (docs-only, +7/-3):

  • Adds a :::caution callout at the top of the Lucia section flagging the deprecation, linking the announcement, and pointing at the replacement auth_session.ts.
  • Replaces the three dead deep-links with verified working targets:
    • code/auth_session.ts (200) as the sessions-API reference,
    • the Lucia repository (200) for cookie guidance,
    • the Auth Book (200) for OAuth patterns.
  • The four example-repository links under "Examples" all return 200 and are untouched.

Every replacement URL in this diff was re-checked with a fresh curl probe after the edit - all 200.

No code or behavior is affected.

…note

Lucia was deprecated in March 2025 (https://pilcrowonpaper.com/blog/18). The three subpages linked from the Astro authentication guide — /sessions/basic-api/, /sessions/cookies/astro, and /tutorials/github-oauth/astro — all return HTTP 404 as of October 2026. Verified via direct curl probes.

This commit:
- Adds a :::caution callout at the top of the Lucia section flagging the deprecation, the announcement, and the working landing page.
- Replaces the three broken deep-links with verified working targets: the single-file reference auth_session.ts, the Lucia repository (for cookie guidance), and the Auth Book for OAuth patterns.
- Leaves the four example repository links untouched (all return 200).

Discovered while sweeping verified-broken doc URLs in the withastro/docs repository. Docs-only change; no code or behavior affected.
@astrobot-houston

Copy link
Copy Markdown
Contributor

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any broken links you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. Once a maintainer approves our workflows, you’ll be able to see a preview of your changes on Cloudflare 🥳.

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

@astrobot-houston

Copy link
Copy Markdown
Contributor

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the tracking.ignoredKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
guides/authentication.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@ArmandPhilippot

Copy link
Copy Markdown
Member

Thanks for catching that, but this isn't the right fix:

  • if the subpages are 404, the fix is not to show a caution but to remove them
  • there is no reasons to use steps here for a single file; there is no separate cookie guidance in the Lucia repository
  • the Auth book lives in another domain

In our CONTRIBUTING.md guide, we ask you to open an issue first for this kind of changes; this isn't an "obvious" fix. So, I'm going to close this. Feel free to open an issue instead.

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.

3 participants