fix: show 404 page if /api/v2/meta/page/{page} can't find a file

Foxnouns.Backend/Controllers/MetaController.cs

@@ -58,8 +58,15 @@ public partial class MetaController(Config config) : ApiControllerBase
         }
 
         string path = Path.Join(Directory.GetCurrentDirectory(), "static-pages", $"{page}.md");
-        string text = await System.IO.File.ReadAllTextAsync(path, ct);
-        return Ok(text);
+        try
+        {
+            string text = await System.IO.File.ReadAllTextAsync(path, ct);
+            return Ok(text);
+        }
+        catch (FileNotFoundException)
+        {
+            throw new ApiError.NotFound("Page not found", code: ErrorCode.PageNotFound);
+        }
     }
 
     [HttpGet("/api/v2/coffee")]

Foxnouns.Backend/ExpectedError.cs

@@ -164,6 +164,7 @@ public enum ErrorCode
     GenericApiError,
     UserNotFound,
     MemberNotFound,
+    PageNotFound,
     AccountAlreadyLinked,
     LastAuthMethod,
     InvalidReportTarget,

Foxnouns.Frontend/src/hooks.server.ts

@@ -21,12 +21,8 @@ Sentry.init({
 });
 
 export const handleError: HandleServerError = async ({ error, status, message }) => {
-	// as far as i know, sentry IDs are just UUIDs with the dashes removed. use those here as well
-	let id = crypto.randomUUID().replaceAll("-", "");
-
 	if (error instanceof ApiError) {
 		return {
-			error_id: id,
 			status: error.raw?.status || status,
 			message: error.raw?.message || "Unknown error",
 			code: error.code,
@@ -34,11 +30,11 @@ export const handleError: HandleServerError = async ({ error, status, message })
 	}
 
 	if (status >= 400 && status <= 499) {
-		return { error_id: id, status, message, code: ErrorCode.GenericApiError };
+		return { status, message, code: ErrorCode.GenericApiError };
 	}
 
 	// client errors and backend API errors just clog up sentry, so we don't send those.
-	id = Sentry.captureException(error, {
+	const id = Sentry.captureException(error, {
 		mechanism: {
 			type: "sveltekit",
 			handled: false,

Foxnouns.Frontend/src/lib/api/error.ts

@@ -43,6 +43,7 @@ export enum ErrorCode {
 	MemberNotFound = "MEMBER_NOT_FOUND",
 	AccountAlreadyLinked = "ACCOUNT_ALREADY_LINKED",
 	LastAuthMethod = "LAST_AUTH_METHOD",
+	PageNotFound = "PAGE_NOT_FOUND",
 	// This code isn't actually returned by the API
 	Non204Response = "(non 204 response)",
 }

Foxnouns.Frontend/src/lib/errorCodes.ts

@@ -29,6 +29,8 @@ export default function errorDescription(t: TranslateFn, code: ErrorCode): strin
 			return t("error.account-already-linked");
 		case ErrorCode.LastAuthMethod:
 			return t("error.last-auth-method");
+		case ErrorCode.PageNotFound:
+			return t("error.page-not-found");
 		case ErrorCode.Non204Response:
 			return t("error.generic-error");
 	}

Foxnouns.Frontend/src/lib/i18n/locales/en.json

@@ -120,7 +120,8 @@
 		"400-description": "Something went wrong with your request. This error should never land you on this page, so it's probably a bug.",
 		"500-description": "Something went wrong on the server. Please try again later.",
 		"unknown-status-description": "Something went wrong, but we're not sure what. Please try again.",
-		"error-id": "If you report this error to the developers, please give them this ID:"
+		"error-id": "If you report this error to the developers, please give them this ID:",
+		"page-not-found": "No page exists at this URL."
 	},
 	"settings": {
 		"general-information-tab": "General information",