Skip to content

Replace noindex with rel=canonical - #1341

Merged
tunetheweb merged 1 commit into
mainfrom
tunetheweb-patch-1
Sep 20, 2026
Merged

tunetheweb merged 1 commit into
mainfrom
tunetheweb-patch-1

Conversation

@tunetheweb

Copy link
Copy Markdown
Member

For static sites:

Astro.url.hostname === 'httparchive.org'

Is evaluated at build time and so always is true. Meaning this is not reevaluated for staging sites and the like.

Let's use rel=canonical instead (which also covers production sites we don't want indexed like httparchive.web.app, wordpress.httparchive.org...etc).

@github-actions

Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit c023ae3):

https://httparchive--pr1341-tunetheweb-patch-1-nbqrke2v.web.app

(expires Sun, 27 Sep 2026 21:25:40 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 1069cba7974889c25bacf4f0a19ddb6735592729

@tunetheweb
tunetheweb merged commit 060418b into main Sep 20, 2026
5 checks passed
@tunetheweb
tunetheweb deleted the tunetheweb-patch-1 branch September 20, 2026 21:39
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.

2 participants