feat: Add more information to errors if we think cookies are missing

guardian · Nov 21, 2024 · 18488bb · 18488bb
1 parent 71afae6
commit 18488bb
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 6 deletions.
diff --git a/src/client/components/GatewayErrorSummary.tsx b/src/client/components/GatewayErrorSummary.tsx
@@ -18,18 +18,31 @@ interface GateWayErrorSummaryProps
 
 export const GatewayErrorSummary = (props: GateWayErrorSummaryProps) => {
 	const structuredError = asStructuredError(props.gatewayError);
-	const fullContext =
-		props.shortRequestId && structuredError?.severity !== 'BAU'
+	const fullContext = [
+		props.context,
+		...(structuredError?.severity === 'CSRF'
+			? [
+					<p>
+						If the problem persists, please check if you have cookies enabled.
+						You can find details on how to enable cookies in our{' '}
+						<a href="https://www.theguardian.com/info/cookies#how-to-manage-cookies-at-the-guardian">
+							Cookies FAQ
+						</a>
+						.
+					</p>,
+				]
+			: []),
+		...(props.shortRequestId && structuredError?.severity !== 'BAU'
 			? [
-					props.context,
 					<p
 						css={[errorContextSpacing, errorContextLastTypeSpacing]}
 						key={'requestId'}
 					>
 						Request ID: {props.shortRequestId}
 					</p>,
 				]
-			: props.context;
+			: []),
+	];
 
 	return (
 		<ErrorSummary

diff --git a/src/server/lib/renderer.tsx b/src/server/lib/renderer.tsx
@@ -86,7 +86,10 @@ const clientStateFromRequestStateLocals = ({
 			...clientState,
 			pageData: {
 				...clientState.pageData,
-				formError: CsrfErrors.CSRF_ERROR,
+				formError: {
+					message: CsrfErrors.CSRF_ERROR,
+					severity: 'CSRF',
+				},
 			},
 		};
 	}

diff --git a/src/shared/model/Errors.ts b/src/shared/model/Errors.ts
@@ -92,7 +92,7 @@ export enum SubscribeErrors {
 
 export interface StructuredGatewayError {
 	message: string;
-	severity: 'BAU' | 'UNEXPECTED';
+	severity: 'BAU' | 'CSRF' | 'UNEXPECTED';
 }
 
 export type GatewayError = StructuredGatewayError | string;