Fix URLs in <head> (#90)

kytta · Jan 26, 2025 · e38d3ed · e38d3ed
2 parents e8e16c5 + a514ac5
commit e38d3ed
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -24,7 +24,11 @@ and this project adheres to
 
 ### Fixed
 
+- `canonical` URL is now always the site's root URL
+  - this fixes potential data leakage in Google Search Console
 - Share2Fedi can be deployed to Deno Deploy again
+- OpenGraph tags are now fixed
+  - `url` used to point to the image URL, instead of the sites URL
 
 ### Changed
 

diff --git a/src/layouts/layout.astro b/src/layouts/layout.astro
@@ -36,7 +36,7 @@ import package_ from "../../package.json";
 		/>
 		<link
 			rel="canonical"
-			href={Astro.url}
+			href={Astro.url.origin}
 		/>
 		<meta
 			property="og:title"
@@ -48,11 +48,11 @@ import package_ from "../../package.json";
 		/>
 		<meta
 			property="og:image"
-			content="/banner.png"
+			content={new URL("/banner.png", Astro.url)}
 		/>
 		<meta
 			property="og:url"
-			content={`${Astro.url}banner.png`}
+			content={Astro.url.origin}
 		/>
 
 		<link