Fixed errors when a server action was executed without reCAPTCHA

app/[locale]/authentication/components/oauth.tsx

@@ -87,7 +87,7 @@ export default function OAuthForm()
 				{
 					startTransition( () =>
 					{
-						serverAction( signInAction, formData );
+						serverAction( signInAction, formData, messages );
 					} );
 				}}
 			>
@@ -125,7 +125,7 @@ export default function OAuthForm()
 				{
 					startTransition( () =>
 					{
-						serverAction( signInAction, formData );
+						serverAction( signInAction, formData, messages );
 					} );
 				}}
 			>

app/[locale]/authentication/components/signin.tsx

@@ -111,7 +111,7 @@ export default function SignInForm()
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( signInAction, formData );
+						serverAction( signInAction, formData, messages );
 					} );
 				}}
 				className="space-y-6"

app/[locale]/authentication/components/signup.tsx

@@ -103,7 +103,7 @@ export default function SignUpForm()
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( signUpAction, formData );
+						serverAction( signUpAction, formData, messages );
 					} );
 				}}
 				className="space-y-6"

app/[locale]/components/cookie-consent.tsx

@@ -4,7 +4,6 @@
 
 "use client";
 
-import { toast } from "sonner";
 import { useMessages } from "next-intl";
 import { usePathname } from "next/navigation";
 import { GoogleTagManager } from "@next/third-parties/google";
@@ -138,18 +137,6 @@ export default function CookieConsent()
 						process.env.NEXT_PUBLIC_ANALYTICS_ENABLED === "true"
 					);
 				}
-
-				// Google reCAPTCHA.
-				if (
-					!cookie.categories.includes( "security" )
-					&& process.env.NEXT_PUBLIC_RECAPTCHA_ENABLED === "true"
-				)
-				{
-					toast.warning( messages.form.errors.recaptcha_failed, {
-						duration: 10000,
-						description: messages.form.errors.recaptcha_error
-					} );
-				}
 			}
 		} );
 	}, [ pathname, messages ] );

app/[locale]/components/notification.tsx

@@ -135,7 +135,7 @@ export default function Notifications()
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const state = await serverAction( updateReadState, new FormData() );
+		const state = await serverAction( updateReadState, new FormData(), formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );

app/[locale]/components/user-menu.tsx

@@ -28,7 +28,8 @@ export default function UserMenu( { session }: { session: Session } )
 {
 	// Déclaration des variables d'état.
 	const router = useRouter();
-	const messages = useTranslations( "header" );
+	const formMessages = useTranslations( "form" );
+	const headerMessages = useTranslations( "header" );
 	const [ isOpen, setOpen ] = useState( false );
 
 	// Déclaration des constantes.
@@ -141,9 +142,7 @@ export default function UserMenu( { session }: { session: Session } )
 							isAdmin ? "text-destructive" : "text-primary"
 						}`}
 					>
-						{isAdmin
-							? messages( "admin_account" )
-							: messages( "user_account" )}
+						{isAdmin ? headerMessages( "admin_account" ) : headerMessages( "user_account" )}
 					</p>
 				</DropdownMenuLabel>
 
@@ -154,7 +153,7 @@ export default function UserMenu( { session }: { session: Session } )
 				<DropdownMenuGroup>
 					<Link href="/dashboard">
 						<DropdownMenuItem>
-							{messages( "dashboard" )}
+							{headerMessages( "dashboard" )}
 
 							<DropdownMenuShortcut>
 								ALT/⌥ + D
@@ -164,7 +163,7 @@ export default function UserMenu( { session }: { session: Session } )
 
 					<Link href="/settings">
 						<DropdownMenuItem>
-							{messages( "settings" )}
+							{headerMessages( "settings" )}
 
 							<DropdownMenuShortcut>
 								ALT/⌥ + S
@@ -179,10 +178,10 @@ export default function UserMenu( { session }: { session: Session } )
 				<DropdownMenuItem
 					onClick={() =>
 					{
-						serverAction( signOutAccount, new FormData() );
+						serverAction( signOutAccount, new FormData(), formMessages );
 					}}
 				>
-					{messages( "logout" )}
+					{headerMessages( "logout" )}
 					<DropdownMenuShortcut>ALT/⌥ + L</DropdownMenuShortcut>
 				</DropdownMenuItem>
 			</DropdownMenuContent>

app/[locale]/dashboard/components/file-history.tsx

@@ -70,7 +70,7 @@ export default function FileHistory( {
 
 		// Envoi de la requête au serveur et
 		//  attente de la réponse.
-		const data = await serverAction( restoreVersion, form );
+		const data = await serverAction( restoreVersion, form, formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );

app/[locale]/dashboard/components/file-upload.tsx

@@ -379,7 +379,7 @@ export default function FileUpload( {
 							// Exécution de l'action côté serveur.
 							startTransition( () =>
 							{
-								serverAction( uploadAction, formData );
+								serverAction( uploadAction, formData, formMessages );
 							} );
 						}}
 					>
@@ -567,7 +567,9 @@ export default function FileUpload( {
 													name={field.name}
 													checked={field.value}
 													disabled={isPending}
-													onCheckedChange={( value ) =>
+													onCheckedChange={(
+														value
+													) =>
 													{
 														if ( value )
 														{
@@ -718,7 +720,8 @@ export default function FileUpload( {
 													disabled={( date ) => date > oneYear
 														|| date < today}
 													onSelect={( value ) => field.onChange(
-														value?.toISOString() ?? ""
+														value?.toISOString()
+																?? ""
 													)}
 													className="rounded-md border"
 													initialFocus

app/[locale]/dashboard/components/row-actions.tsx

@@ -97,7 +97,11 @@ export default function RowActions( {
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const files = ( await serverAction( changeFileStatus, form ) ) as string[];
+		const files = ( await serverAction(
+			changeFileStatus,
+			form,
+			formMessages
+		) ) as string[];
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -163,7 +167,11 @@ export default function RowActions( {
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const files = ( await serverAction( changeFileStatus, form ) ) as string[];
+		const files = ( await serverAction(
+			changeFileStatus,
+			form,
+			formMessages
+		) ) as string[];
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -221,7 +229,11 @@ export default function RowActions( {
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const files = ( await serverAction( renameFile, form ) ) as string[];
+		const files = ( await serverAction(
+			renameFile,
+			form,
+			formMessages
+		) ) as string[];
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -277,7 +289,11 @@ export default function RowActions( {
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const files = ( await serverAction( deleteFile, form ) ) as string[];
+		const files = ( await serverAction(
+			deleteFile,
+			form,
+			formMessages
+		) ) as string[];
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -331,7 +347,7 @@ export default function RowActions( {
 
 		// Envoi de la requête au serveur et
 		//  traitement de la réponse.
-		const state = await serverAction( deleteSharedUser, form );
+		const state = await serverAction( deleteSharedUser, form, formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );

app/[locale]/dashboard/components/share-manager.tsx

@@ -93,7 +93,7 @@ export default function ShareManager( {
 
 		// Envoi de la requête au serveur et
 		//  attente de la réponse.
-		const state = await serverAction( addSharedUser, form );
+		const state = await serverAction( addSharedUser, form, formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -145,7 +145,7 @@ export default function ShareManager( {
 
 		// Envoi de la requête au serveur et
 		//  attente de la réponse.
-		const state = await serverAction( updateSharedUser, form );
+		const state = await serverAction( updateSharedUser, form, formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );
@@ -184,7 +184,7 @@ export default function ShareManager( {
 
 		// Envoi de la requête au serveur et
 		//  attente de la réponse.
-		const state = await serverAction( deleteSharedUser, form );
+		const state = await serverAction( deleteSharedUser, form, formMessages );
 
 		// Fin de l'état de chargement.
 		setLoading( false );

app/[locale]/settings/components/appearance.tsx

@@ -135,7 +135,7 @@ export default function Appearance( { session }: { session: Session } )
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( updateAction, formData );
+						serverAction( updateAction, formData, messages );
 					} );
 				}}
 				className="space-y-8"

app/[locale]/settings/components/issue.tsx

@@ -116,7 +116,7 @@ export default function Account()
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( updateAction, formData );
+						serverAction( updateAction, formData, messages );
 					} );
 				}}
 				className="space-y-8"

app/[locale]/settings/components/notifications.tsx

@@ -110,7 +110,7 @@ export default function Notifications( { session }: { session: Session } )
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( updateAction, formData );
+						serverAction( updateAction, formData, messages );
 					} );
 				}}
 				className="space-y-8"

app/[locale]/settings/components/privacy.tsx

@@ -113,7 +113,7 @@ export default function Privacy()
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( deleteAction, formData );
+						serverAction( deleteAction, formData, formMessages );
 					} );
 				}}
 				className="space-y-8"

app/[locale]/settings/components/storage.tsx

@@ -99,7 +99,7 @@ export default function Storage( { session }: { session: Session } )
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( updateAction, formData );
+						serverAction( updateAction, formData, messages );
 					} );
 				}}
 				className="space-y-8"

app/[locale]/settings/components/user.tsx

@@ -156,7 +156,7 @@ export default function User( { session }: { session: Session } )
 					// Exécution de l'action côté serveur.
 					startTransition( () =>
 					{
-						serverAction( updateAction, formData );
+						serverAction( updateAction, formData, messages );
 					} );
 				}}
 				className="space-y-8"

utilities/recaptcha.ts

@@ -5,9 +5,12 @@
 
 "use client";
 
+import { toast } from "sonner";
+
 export default async function serverAction(
 	action: ( payload: FormData ) => void,
-	formData: FormData
+	formData: FormData,
+	messages: ( key: string ) => string
 )
 {
 	// On vérifie d'abord si le service reCAPTCHA est présent ou non.
@@ -16,9 +19,33 @@ export default async function serverAction(
 	//   ne sont pas encore chargés dans le navigateur de l'utilisateur.
 	if ( typeof window.grecaptcha === "undefined" )
 	{
-		// Dans ce cas, on traite la requête comme si le service reCAPTCHA n'était
-		//  pas activé et on laisse le serveur répondre à l'utilisateur.
-		return action( formData );
+		// Premier cas de figure : le serveur utilise reCAPTCHA mais le client
+		//  n'a pas encore chargé les services de Google reCAPTCHA ou ils ont été
+		//  explicitement désactivés par l'utilisateur.
+		if ( process.env.NEXT_PUBLIC_RECAPTCHA_ENABLED === "true" )
+		{
+			toast.error( messages( "errors.recaptcha_failed" ), {
+				duration: 10000,
+				description: messages( "errors.recaptcha_error" )
+			} );
+
+			return false;
+		}
+
+		// Deuxième cas de figure : le serveur n'utilise pas reCAPTCHA.
+		//  On exécute alors l'action côté serveur sans vérification.
+		try
+		{
+			return action( formData );
+		}
+		catch
+		{
+			toast.error( messages( "errors.internal_error" ), {
+				description: messages( "errors.server_error" )
+			} );
+
+			return false;
+		}
 	}
 
 	// On créé après une promesse afin de gérer le chargement des services de