From 85b84e006747bea5b875a536005213a4b7b09e9c Mon Sep 17 00:00:00 2001
From: Benoit Jeaurond <benoit@jeaurond.dev>
Date: Fri, 14 Jun 2024 20:46:37 -0400
Subject: [PATCH] lint: address more issues

---
 backend/cli/src/persistence.rs                |  4 +-
 backend/database/src/validators.rs            |  2 +-
 backend/server/src/routes/openapi.rs          |  6 +-
 frontend/src/api/ignoreLists.ts               |  2 +-
 frontend/src/api/providers.ts                 |  4 +-
 frontend/src/api/sources.ts                   |  2 +-
 frontend/src/api/users.ts                     |  2 +-
 frontend/src/components/api-tokens-table.tsx  |  2 +-
 frontend/src/components/empty.tsx             | 44 +++++++------
 frontend/src/components/error.tsx             |  1 +
 .../src/components/forms/list-edit-create.tsx |  2 +-
 .../components/forms/provider-edit-create.tsx |  2 +-
 .../components/forms/source-edit-create.tsx   |  2 +-
 .../src/components/layouts/authenticated.tsx  |  2 +-
 .../src/components/layouts/authentication.tsx |  4 ++
 frontend/src/components/nav.tsx               |  4 +-
 frontend/src/components/raw-content.tsx       | 65 ++++++-------------
 frontend/src/components/request-data-view.tsx | 14 ++--
 .../search-results/provider-search-result.tsx |  3 +
 .../search-results/source-search-result.tsx   |  3 +
 frontend/src/components/secrets-table.tsx     |  2 +-
 frontend/src/components/sources/index.tsx     | 11 ++--
 frontend/src/i18n/en_CA.json                  |  6 ++
 frontend/src/i18n/fr_CA.json                  |  6 ++
 frontend/src/i18n/tools/unusedKeys.ts         |  1 +
 frontend/src/lib/auth.ts                      | 12 ++--
 frontend/src/routes/docs.tsx                  |  1 +
 frontend/src/routes/history/route.tsx         |  2 +-
 frontend/src/routes/sources/$slug.tsx         |  2 +-
 29 files changed, 114 insertions(+), 99 deletions(-)

diff --git a/backend/cli/src/persistence.rs b/backend/cli/src/persistence.rs
index ea16296..a274e88 100644
--- a/backend/cli/src/persistence.rs
+++ b/backend/cli/src/persistence.rs
@@ -1,6 +1,6 @@
 use std::{
     env::current_dir,
-    fs::{create_dir, File},
+    fs::{create_dir_all, File},
     io::Write,
     path::PathBuf,
 };
@@ -16,7 +16,7 @@ impl SourceFileWriter {
         let mut directory = current_dir().unwrap();
         directory.push(Local::now().naive_local().to_string());
 
-        create_dir(&directory).unwrap();
+        create_dir_all(&directory).unwrap();
 
         Self { directory }
     }
diff --git a/backend/database/src/validators.rs b/backend/database/src/validators.rs
index 489d144..a90fe50 100644
--- a/backend/database/src/validators.rs
+++ b/backend/database/src/validators.rs
@@ -15,7 +15,7 @@ fn validate_hash(data: &str, expected_hash: Hash) -> bool {
 
 #[instrument]
 fn validate_domain(data: &str) -> bool {
-    data.split('.').count() > 1 && data.split('.').all(|section| !section.is_empty())
+    data.contains('.') && data.split('.').all(|section| !section.is_empty())
 }
 
 impl Indicator {
diff --git a/backend/server/src/routes/openapi.rs b/backend/server/src/routes/openapi.rs
index 6c6ca7a..204cda8 100644
--- a/backend/server/src/routes/openapi.rs
+++ b/backend/server/src/routes/openapi.rs
@@ -245,10 +245,10 @@ impl Modify for SecurityAddon {
         );
         openapi.components = Some(components);
 
-        let scopes: [&str; 0] = [];
+        let scopes: Vec<String> = Vec::with_capacity(0);
         let data = vec![
-            SecurityRequirement::new("api_key", scopes),
-            SecurityRequirement::new("basic_auth", scopes),
+            SecurityRequirement::new("api_key", &scopes),
+            SecurityRequirement::new("basic_auth", &scopes),
         ];
         let unauthorized_response: RefOr<_> = ResponseBuilder::new()
             .description("Need to provide valid authentication")
diff --git a/frontend/src/api/ignoreLists.ts b/frontend/src/api/ignoreLists.ts
index a2aadd7..63ae7e4 100644
--- a/frontend/src/api/ignoreLists.ts
+++ b/frontend/src/api/ignoreLists.ts
@@ -32,7 +32,7 @@ export function ignoreListSlugQueryOptions(slug: string) {
   return queryOptions({
     queryKey: ["ignoreLists", "slugs", slug],
     queryFn: async ({ signal }) =>
-      fetcher.get<string>(`/ignoreLists/slugs/${slug}`, {
+      await fetcher.get<string>(`/ignoreLists/slugs/${slug}`, {
         signal,
       }),
   });
diff --git a/frontend/src/api/providers.ts b/frontend/src/api/providers.ts
index f2f1ff9..a706c32 100644
--- a/frontend/src/api/providers.ts
+++ b/frontend/src/api/providers.ts
@@ -30,7 +30,7 @@ export function providerSlugQueryOptions(slug: string) {
   return queryOptions({
     queryKey: ["providers", "slugs", slug],
     queryFn: async ({ signal }) =>
-      fetcher.get<string>(`/providers/slugs/${slug}`, {
+      await fetcher.get<string>(`/providers/slugs/${slug}`, {
         signal,
       }),
   });
@@ -42,7 +42,7 @@ export function providerQueryOptions<T extends undefined | string>(
   return queryOptions({
     queryKey: ["providers", providerId],
     queryFn: async ({ signal }) => {
-      if (providerId == undefined) {
+      if (!providerId) {
         return null;
       }
 
diff --git a/frontend/src/api/sources.ts b/frontend/src/api/sources.ts
index 5c0d7eb..36f865f 100644
--- a/frontend/src/api/sources.ts
+++ b/frontend/src/api/sources.ts
@@ -32,7 +32,7 @@ export function sourceSlugQueryOptions(slug: string) {
   return queryOptions({
     queryKey: ["sources", "slugs", slug],
     queryFn: async ({ signal }) =>
-      fetcher.get<string>(`/sources/slugs/${slug}`, {
+      await fetcher.get<string>(`/sources/slugs/${slug}`, {
         signal,
       }),
   });
diff --git a/frontend/src/api/users.ts b/frontend/src/api/users.ts
index 01700f2..20247fe 100644
--- a/frontend/src/api/users.ts
+++ b/frontend/src/api/users.ts
@@ -25,7 +25,7 @@ export function userQueryOptions<T extends undefined | string>(userId: T) {
   return queryOptions({
     queryKey: ["users", userId],
     queryFn: async ({ signal }) => {
-      if (userId == undefined) {
+      if (!userId) {
         return null;
       }
 
diff --git a/frontend/src/components/api-tokens-table.tsx b/frontend/src/components/api-tokens-table.tsx
index 4260a06..6caed9e 100644
--- a/frontend/src/components/api-tokens-table.tsx
+++ b/frontend/src/components/api-tokens-table.tsx
@@ -216,7 +216,7 @@ const ApiTokensTable: React.FC<Props> = ({ apiTokens }) => {
                   <TableHead>
                     <Trans id="expires.at" />
                   </TableHead>
-                  <TableHead style={{ width: 110 }}></TableHead>
+                  <TableHead style={{ width: 110 }} />
                 </TableRow>
               </TableHeader>
               <TableBody className="text-xs">
diff --git a/frontend/src/components/empty.tsx b/frontend/src/components/empty.tsx
index 6177870..c01160a 100644
--- a/frontend/src/components/empty.tsx
+++ b/frontend/src/components/empty.tsx
@@ -1,5 +1,5 @@
 import { cn } from "@/lib/utils";
-import { TransId } from "@/i18n";
+import { TransId, useTranslation } from "@/i18n";
 import { Trans } from "@/components";
 
 interface Props {
@@ -18,24 +18,30 @@ const Empty: React.FC<Props> = ({
   imageWidth = 500,
   extra,
   className,
-}) => (
-  <div
-    className={cn(
-      "flex flex-1 flex-col items-center justify-center gap-2 self-center justify-self-center p-4",
-      className,
-    )}
-  >
-    {image && <img src={image} width={imageWidth} />}
-    <div className="flex flex-col items-center text-center">
-      <h2 className="text-lg font-semibold">
-        <Trans id={title} />
-      </h2>
-      <p className="text-sm">
-        <Trans id={description} />
-      </p>
+}) => {
+  const { t } = useTranslation();
+
+  return (
+    <div
+      className={cn(
+        "flex flex-1 flex-col items-center justify-center gap-2 self-center justify-self-center p-4",
+        className,
+      )}
+    >
+      {image && (
+        <img src={image} alt={t("empty.image.alt")} width={imageWidth} />
+      )}
+      <div className="flex flex-col items-center text-center">
+        <h2 className="text-lg font-semibold">
+          <Trans id={title} />
+        </h2>
+        <p className="text-sm">
+          <Trans id={description} />
+        </p>
+      </div>
+      {extra}
     </div>
-    {extra}
-  </div>
-);
+  );
+};
 
 export default Empty;
diff --git a/frontend/src/components/error.tsx b/frontend/src/components/error.tsx
index 9b27c14..d829099 100644
--- a/frontend/src/components/error.tsx
+++ b/frontend/src/components/error.tsx
@@ -41,6 +41,7 @@ const ErrorComponent: React.FC<Props> = ({ info, error }) => {
       <div className="hidden h-[20rem] w-[20rem] flex-col gap-6 lg:flex xl:h-[30rem] xl:w-[30rem]">
         <img
           src={Image}
+          alt={t("this.is.fine.image.alt")}
           className="h-fit w-fit rounded-xl border border-black object-cover text-white shadow-xl dark:border-border"
         />
 
diff --git a/frontend/src/components/forms/list-edit-create.tsx b/frontend/src/components/forms/list-edit-create.tsx
index 8f4dce2..089d989 100644
--- a/frontend/src/components/forms/list-edit-create.tsx
+++ b/frontend/src/components/forms/list-edit-create.tsx
@@ -480,7 +480,7 @@ const ListEditCreate: React.FC<Props> = ({
                           <TableHead>
                             <Trans id="kind" />
                           </TableHead>
-                          <TableHead style={{ width: 50 }}></TableHead>
+                          <TableHead style={{ width: 50 }} />
                         </TableRow>
                       </TableHeader>
                       <TableBody>
diff --git a/frontend/src/components/forms/provider-edit-create.tsx b/frontend/src/components/forms/provider-edit-create.tsx
index ea48a7e..2b00eb4 100644
--- a/frontend/src/components/forms/provider-edit-create.tsx
+++ b/frontend/src/components/forms/provider-edit-create.tsx
@@ -367,7 +367,7 @@ const ProviderEditCreate: React.FC<Props> = ({
                       <div className="flex gap-2">
                         <img
                           src={field.value ?? undefined}
-                          alt={t("favicon")}
+                          alt={t("provider.favicon.alt")}
                           style={{ imageRendering: "pixelated" }}
                           className={cn(
                             "h-8 w-8 rounded border shadow",
diff --git a/frontend/src/components/forms/source-edit-create.tsx b/frontend/src/components/forms/source-edit-create.tsx
index 245667c..c41816e 100644
--- a/frontend/src/components/forms/source-edit-create.tsx
+++ b/frontend/src/components/forms/source-edit-create.tsx
@@ -496,7 +496,7 @@ const SourceEditCreate: React.FC<Props> = ({
                       <div className="flex gap-2">
                         <img
                           src={field.value ?? undefined}
-                          alt="favicon"
+                          alt={t("source.favicon.alt")}
                           style={{ imageRendering: "pixelated" }}
                           className={cn(
                             "h-8 w-8 rounded border shadow",
diff --git a/frontend/src/components/layouts/authenticated.tsx b/frontend/src/components/layouts/authenticated.tsx
index c2dc7e9..dc91a6c 100644
--- a/frontend/src/components/layouts/authenticated.tsx
+++ b/frontend/src/components/layouts/authenticated.tsx
@@ -142,7 +142,7 @@ export const Layout: React.FC = () => {
               },
             ]}
           />
-          {userHasAnyRoles(user!, ["request_create", "request_view"]) && (
+          {userHasAnyRoles(user, ["request_create", "request_view"]) && (
             <div className={cn("mx-4", isCollapsed && "mx-2")}>
               <Separator />
             </div>
diff --git a/frontend/src/components/layouts/authentication.tsx b/frontend/src/components/layouts/authentication.tsx
index 9ddea63..98a8ebd 100644
--- a/frontend/src/components/layouts/authentication.tsx
+++ b/frontend/src/components/layouts/authentication.tsx
@@ -3,11 +3,13 @@ import { TrainFrontTunnel } from "lucide-react";
 
 import { Layouts, Trans } from "@/components";
 import { getRandomBackground } from "@/assets";
+import { useTranslation } from "@/i18n";
 
 export const Authentication: React.FC<React.PropsWithChildren> = ({
   children,
 }) => {
   const bg = useMemo(() => getRandomBackground(), []);
+  const { t } = useTranslation();
 
   return (
     <div className="relative grid h-[800px] flex-col items-center justify-center lg:max-w-none lg:grid-cols-5 lg:ps-4">
@@ -16,11 +18,13 @@ export const Authentication: React.FC<React.PropsWithChildren> = ({
           <img
             id="lowres-login-img"
             src={bg.small}
+            alt={t("login.image.blurry.alt")}
             className="absolute h-full flex-1 object-cover blur-xl"
           />
           <img
             id="login-img"
             src={bg.big}
+            alt={t("login.image.alt")}
             className="absolute h-full flex-1 object-cover opacity-0 blur-xl transition duration-1000 ease-out"
             style={{ transitionProperty: "filter" }}
             onLoad={() => {
diff --git a/frontend/src/components/nav.tsx b/frontend/src/components/nav.tsx
index 071a0e5..bcf0336 100644
--- a/frontend/src/components/nav.tsx
+++ b/frontend/src/components/nav.tsx
@@ -59,7 +59,7 @@ const Nav: React.FC<NavProps> = ({ links, isCollapsed }) => {
   const user = useAtomValue(userAtom);
 
   const canViewAnyLinks = links.some(
-    (link) => !("roles" in link) || userHasRoles(user!, link.roles),
+    (link) => !("roles" in link) || userHasRoles(user, link.roles),
   );
 
   if (!canViewAnyLinks) {
@@ -74,7 +74,7 @@ const Nav: React.FC<NavProps> = ({ links, isCollapsed }) => {
       <nav className="grid gap-1 px-2 group-[[data-collapsed=true]]:justify-center group-[[data-collapsed=true]]:px-2">
         {links
           .filter(
-            (link) => !("roles" in link) || userHasRoles(user!, link.roles),
+            (link) => !("roles" in link) || userHasRoles(user, link.roles),
           )
           .map((link, index) =>
             isCollapsed ? (
diff --git a/frontend/src/components/raw-content.tsx b/frontend/src/components/raw-content.tsx
index a4f4bae..e1c4477 100644
--- a/frontend/src/components/raw-content.tsx
+++ b/frontend/src/components/raw-content.tsx
@@ -1,54 +1,31 @@
-import { useMemo } from "react";
-
 import CopyButton from "@/components/copy-button";
 import { cn } from "@/lib/utils";
 
 const RawContent: React.FC<{
   content?: string;
-  iframe?: boolean;
-  dangerouslyShowContent?: boolean;
   className?: string;
-}> = ({ content, iframe, dangerouslyShowContent, className }) => {
-  const cleanedContent = useMemo(() => {
-    return dangerouslyShowContent
-      ? content
-      : content
-          ?.replaceAll(/http/gi, "hxxp")
-          .replaceAll(/src="\/\//gi, 'src="hxxp://');
-  }, [content, dangerouslyShowContent]);
-
-  return (
-    <div
-      className={cn(
-        "dark:border-800 dark:bg-950 relative flex h-full w-full flex-1 overflow-hidden rounded border bg-muted shadow-md ring-white",
-        className,
-      )}
-    >
-      <div className="flex h-full max-h-[400px] min-h-[1.5rem] flex-1 text-xs dark:bg-black/50">
-        {iframe && (
-          <iframe
-            sandbox=""
-            srcDoc={cleanedContent}
-            className="h-[400px] w-full flex-1 dark:bg-white"
-          />
-        )}
-        {!iframe && (
-          <div
-            style={{
-              fontFamily:
-                'ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace',
-            }}
-            className="flex h-full max-h-[400px] w-full flex-1 overflow-y-scroll whitespace-pre-wrap break-all p-3"
-          >
-            {cleanedContent}
-          </div>
-        )}
-        <div className="absolute bottom-2 right-2">
-          <CopyButton text={content} />
-        </div>
+}> = ({ content, className }) => (
+  <div
+    className={cn(
+      "dark:border-800 dark:bg-950 relative flex h-full w-full flex-1 overflow-hidden rounded border bg-muted shadow-md ring-white",
+      className,
+    )}
+  >
+    <div className="flex h-full max-h-[400px] min-h-[1.5rem] flex-1 text-xs dark:bg-black/50">
+      <div
+        style={{
+          fontFamily:
+            'ui-monospace, SFMono-Regular, Menlo, Monaco, Consolas, "Liberation Mono", "Courier New", monospace',
+        }}
+        className="flex h-full max-h-[400px] w-full flex-1 overflow-y-scroll whitespace-pre-wrap break-all p-3"
+      >
+        {content}
+      </div>
+      <div className="absolute bottom-2 right-2">
+        <CopyButton text={content} />
       </div>
     </div>
-  );
-};
+  </div>
+);
 
 export default RawContent;
diff --git a/frontend/src/components/request-data-view.tsx b/frontend/src/components/request-data-view.tsx
index 07baad9..3acc2ea 100644
--- a/frontend/src/components/request-data-view.tsx
+++ b/frontend/src/components/request-data-view.tsx
@@ -41,10 +41,16 @@ const RequestDataView: React.FC<Props> = ({ isFetching, data }) => {
 
       return [
         sourceWithoutErrors.sort((a, b) => {
-          const aDataLength =
-            a.data == undefined ? 0 : Array.isArray(a.data) ? a.data.length : 1;
-          const bDataLength =
-            b.data == undefined ? 0 : Array.isArray(b.data) ? b.data.length : 1;
+          const aDataLength = !a.data
+            ? 0
+            : Array.isArray(a.data)
+              ? a.data.length
+              : 1;
+          const bDataLength = !b.data
+            ? 0
+            : Array.isArray(b.data)
+              ? b.data.length
+              : 1;
 
           if (aDataLength > bDataLength) {
             return -1;
diff --git a/frontend/src/components/search-results/provider-search-result.tsx b/frontend/src/components/search-results/provider-search-result.tsx
index 78dda0a..642af49 100644
--- a/frontend/src/components/search-results/provider-search-result.tsx
+++ b/frontend/src/components/search-results/provider-search-result.tsx
@@ -8,11 +8,13 @@ import { Badge } from "@/components/ui/badge";
 import { ComponentSearchResultProps } from "@/components/generic-panel-search";
 import config from "@/lib/config";
 import { Trans } from "@/components";
+import { useTranslation } from "@/i18n";
 
 const ProviderSearchResult: React.FC<ComponentSearchResultProps<Provider>> = ({
   data: { slug, name, description, enabled, numSources, url, favicon },
 }) => {
   const [imgHasError, setImgHasError] = useState(false);
+  const { t } = useTranslation();
 
   return (
     <Link
@@ -30,6 +32,7 @@ const ProviderSearchResult: React.FC<ComponentSearchResultProps<Provider>> = ({
               src={
                 favicon ?? `${config.rest_server_base_url}/favicon?url=${url}`
               }
+              alt={t("provider.favicon.alt")}
               style={{ imageRendering: "pixelated" }}
               className={cn(
                 "h-6 w-6 rounded border shadow",
diff --git a/frontend/src/components/search-results/source-search-result.tsx b/frontend/src/components/search-results/source-search-result.tsx
index f08c73c..f7ef395 100644
--- a/frontend/src/components/search-results/source-search-result.tsx
+++ b/frontend/src/components/search-results/source-search-result.tsx
@@ -9,11 +9,13 @@ import { ComponentSearchResultProps } from "@/components/generic-panel-search";
 import config from "@/lib/config";
 import { sourceKindIconMapping } from "@/lib/data";
 import { Trans } from "@/components";
+import { useTranslation } from "@/i18n";
 
 const SourceSearchResult: React.FC<ComponentSearchResultProps<Source>> = ({
   data: { slug, name, description, enabled, providerId, url, favicon, kind },
 }) => {
   const [imgHasError, setImgHasError] = useState(true);
+  const { t } = useTranslation();
 
   const SourceKindIcon = sourceKindIconMapping[kind];
 
@@ -33,6 +35,7 @@ const SourceSearchResult: React.FC<ComponentSearchResultProps<Source>> = ({
               src={
                 favicon ?? `${config.rest_server_base_url}/favicon?url=${url}`
               }
+              alt={t("source.favicon.alt")}
               style={{ imageRendering: "pixelated" }}
               className={cn(
                 "hidden h-8 w-8 rounded border shadow",
diff --git a/frontend/src/components/secrets-table.tsx b/frontend/src/components/secrets-table.tsx
index bb07e95..3798f0a 100644
--- a/frontend/src/components/secrets-table.tsx
+++ b/frontend/src/components/secrets-table.tsx
@@ -144,7 +144,7 @@ const SecretsTable: React.FC<Props> = ({ secrets }) => {
                   >
                     <Trans id="sources" />
                   </TableHead>
-                  <TableHead style={{ width: 80 }}></TableHead>
+                  <TableHead style={{ width: 80 }} />
                 </TableRow>
               </TableHeader>
               <TableBody className="text-xs">
diff --git a/frontend/src/components/sources/index.tsx b/frontend/src/components/sources/index.tsx
index 8b1630e..928ddfe 100644
--- a/frontend/src/components/sources/index.tsx
+++ b/frontend/src/components/sources/index.tsx
@@ -17,7 +17,7 @@ import {
   TooltipTrigger,
 } from "@/components/ui/tooltip";
 import { Editor, Trans } from "@/components";
-import { TransId } from "@/i18n";
+import { TransId, useTranslation } from "@/i18n";
 
 import { PhishTank } from "./phishtank";
 
@@ -39,8 +39,7 @@ export const Source: React.FC<ReqestSSEData> = ({
   errors,
   hasSourceCode,
 }) => {
-  const entryCount =
-    data == undefined ? 0 : Array.isArray(data) ? data.length : 1;
+  const entryCount = !data ? 0 : Array.isArray(data) ? data.length : 1;
   const Content =
     source.name in InnerSource
       ? InnerSource[source.name as keyof typeof InnerSource].Component
@@ -48,6 +47,7 @@ export const Source: React.FC<ReqestSSEData> = ({
   const diff = dayjs(timing?.endedAt).diff(dayjs(timing?.startedAt));
 
   const [imgHasError, setImgHasError] = useState(true);
+  const { t } = useTranslation();
 
   const shouldHaveData = errors.length === 0 && hasSourceCode;
 
@@ -64,6 +64,7 @@ export const Source: React.FC<ReqestSSEData> = ({
         <div className="flex flex-1 items-center gap-4">
           <a href={source.url} target="_blank" rel="noopener noreferrer">
             <img
+              alt={t("source.favicon.alt")}
               src={
                 source.favicon ??
                 `${config.rest_server_base_url}/favicon?url=${source.url}`
@@ -92,8 +93,8 @@ export const Source: React.FC<ReqestSSEData> = ({
         <div className="flex flex-wrap items-center justify-end gap-2 gap-y-1">
           {errors.length !== 0 && (
             <span className="flex flex-col items-end text-xs italic">
-              {errors.map((error, i) => (
-                <span key={i}>
+              {errors.map((error) => (
+                <span key={error.kind}>
                   <Trans id={SourceErrorPreview[error.kind]} />
                 </span>
               ))}
diff --git a/frontend/src/i18n/en_CA.json b/frontend/src/i18n/en_CA.json
index ecd8c23..f45902a 100644
--- a/frontend/src/i18n/en_CA.json
+++ b/frontend/src/i18n/en_CA.json
@@ -112,6 +112,7 @@
   "edit": "Edit",
   "elapsed": "Elapsed",
   "email": "Email",
+  "empty.image.alt": "Illustration representing an empty/loading state",
   "enable.background.task": "Enable background task",
   "enable.caching": "Enable caching",
   "enable.rate.limiter": "Enable rate limiter",
@@ -213,6 +214,8 @@
   "login.alternative.description": "or using email authentication",
   "login.description": "Discover a unified platform where you can query, correlate, and enrich your threat data in real-time, with seamless access to multiple sources and a flexible framework for custom integrations.",
   "login.here": "login here",
+  "login.image.alt": "Random image for login",
+  "login.image.blurry.alt": "Random blurry image for login",
   "login.signup.description": "Don't have an account? {link} to create a new account",
   "logout": "Logout",
   "logs": "Logs",
@@ -260,6 +263,7 @@
   "provider.delete.confirmation.description": "This action cannot be undone. This will permanently {name} as a provider, linked sources will simple not have a provider anymore.",
   "provider.deleted.description": "Provider {name} was deleted successfully and is no longer linked to any sources.",
   "provider.deleted.title": "Provider deleted",
+  "provider.favicon.alt": "Provider favicon",
   "provider.not.found": "Provider {data} not found",
   "provider.saved": "Provider saved",
   "provider.user.id": "Provider user ID",
@@ -319,6 +323,7 @@
   "source.delete.confirmation.description": "This action cannot be undone. This will permanently delete {name} as a source.",
   "source.deleted.description": "Source {name} was deleted successfully.",
   "source.deleted.title": "Source deleted",
+  "source.favicon.alt": "Source favicon",
   "source.interval.limit": "Limit of request per interval",
   "source.interval.number.request": "Number of request during interval",
   "source.interval.reset": "Reset interval",
@@ -352,6 +357,7 @@
   "theme.dark": "Dark theme",
   "theme.light": "Light theme",
   "theme.system": "System theme",
+  "this.is.fine.image.alt": "Animated picture of a dog sitting in a fire",
   "time": "Time",
   "timeout": "Timeout",
   "toggle.theme": "Toggle theme",
diff --git a/frontend/src/i18n/fr_CA.json b/frontend/src/i18n/fr_CA.json
index 484efbd..3dd7d33 100644
--- a/frontend/src/i18n/fr_CA.json
+++ b/frontend/src/i18n/fr_CA.json
@@ -112,6 +112,7 @@
   "edit": "Modifier",
   "elapsed": "Écoulé",
   "email": "Courriel",
+  "empty.image.alt": "Illustration représentant un état de chargement/vide",
   "enable.background.task": "Activer la tâche d'arrière-plan",
   "enable.caching": "Activer la mise en cache",
   "enable.rate.limiter": "Activer le limiteur de taux",
@@ -213,6 +214,8 @@
   "login.alternative.description": "ou en utilisant votre courriel",
   "login.description": "Découvrez une plateforme unifiée où vous pouvez interroger, corréler et enrichir vos données sur les menaces en temps réel, avec un accès transparent à de multiples sources et un cadre flexible pour des intégrations personnalisées",
   "login.here": "connectez-vous ici",
+  "login.image.alt": "Image de l'authentification",
+  "login.image.blurry.alt": "Image floue de l'authentification",
   "login.signup.description": "Vous n'avez pas de compte? {link} pour créer un nouveau compte",
   "logout": "Déconnexion",
   "logs": "Jounaux",
@@ -260,6 +263,7 @@
   "provider.delete.confirmation.description": "Cette action est irrévesible. Cela supprimera le fournisseur {name} de façon permanente, les sources liées n'auront simplement plus de fournisseur.",
   "provider.deleted.description": "Le fournisseur {nom} a été supprimé avec succès et n'est plus lié à aucune source.",
   "provider.deleted.title": "Fournisseur supprimé",
+  "provider.favicon.alt": "Favicône du fournisseur",
   "provider.not.found": "Fournisseur {data} non trouvé",
   "provider.saved": "Fournisseur sauvegardé",
   "provider.user.id": "ID de l'utilisateur du fournisseur",
@@ -319,6 +323,7 @@
   "source.delete.confirmation.description": "Cette action ne peut être annulée. Cette action supprimera en permanence {name} en tant que source.",
   "source.deleted.description": "La source {name} a été supprimée avec succès.",
   "source.deleted.title": "Source supprimée",
+  "source.favicon.alt": "Favicône de la source",
   "source.interval.limit": "Limite de requêtes par intervalle",
   "source.interval.number.request": "Nombre de requêtes au cours de l'intervalle",
   "source.interval.reset": "Intervalle de réinitialisation",
@@ -352,6 +357,7 @@
   "theme.dark": "Thème sombre",
   "theme.light": "Thème lumineux",
   "theme.system": "Thème système",
+  "this.is.fine.image.alt": "Photo animée d'un chien couchant dans un feu",
   "time": "Temps",
   "timeout": "Timeout",
   "toggle.theme": "Changer de thème",
diff --git a/frontend/src/i18n/tools/unusedKeys.ts b/frontend/src/i18n/tools/unusedKeys.ts
index 89167b2..1502c22 100644
--- a/frontend/src/i18n/tools/unusedKeys.ts
+++ b/frontend/src/i18n/tools/unusedKeys.ts
@@ -29,6 +29,7 @@ const findUnusedKeys = (filesContent: string[], searchableKeys: string[]) => {
       unusedKeys,
     );
     console.error("findUnusedKeys check: FAIL\n");
+    process.exit(1);
   } else {
     console.log("All keys in en_CA and fr_CA files are used in the project!");
     console.log("findUnusedKeys check: SUCCESS\n");
diff --git a/frontend/src/lib/auth.ts b/frontend/src/lib/auth.ts
index 52f94e8..b73fb40 100644
--- a/frontend/src/lib/auth.ts
+++ b/frontend/src/lib/auth.ts
@@ -33,8 +33,8 @@ export const beforeLoadAuthenticated: BeforeLoadFn =
           toast.error(
             `You don't have the required roles to access the page ${window.location.pathname}`,
             {
-              id: "no-roles" + missingRoles,
-              description: "Missing roles: " + missingRoles,
+              id: `no-roles${missingRoles}`,
+              description: `Missing roles: ${missingRoles}`,
             },
           );
         }
@@ -60,8 +60,8 @@ export function parseJwt<T>(token: string): T {
   return JSON.parse(jsonPayload) as T;
 }
 
-export const userHasRoles = (user: User, roles?: string[]) =>
-  !roles || roles.every((role) => user.roles.includes(role));
+export const userHasRoles = (user?: User, roles?: string[]) =>
+  !roles || roles.every((role) => user?.roles.includes(role));
 
-export const userHasAnyRoles = (user: User, roles?: string[]) =>
-  !roles || roles.some((role) => user.roles.includes(role));
+export const userHasAnyRoles = (user?: User, roles?: string[]) =>
+  !roles || roles.some((role) => user?.roles.includes(role));
diff --git a/frontend/src/routes/docs.tsx b/frontend/src/routes/docs.tsx
index f6f2699..204371f 100644
--- a/frontend/src/routes/docs.tsx
+++ b/frontend/src/routes/docs.tsx
@@ -195,6 +195,7 @@ const DocsComponents: React.FC = () => {
         <iframe
           sandbox=""
           src={`${config.rest_server_base_url}/docs`}
+          title={t("docs.swagger.ui")}
           width="100%"
           className="min-h-[80vh]"
         />
diff --git a/frontend/src/routes/history/route.tsx b/frontend/src/routes/history/route.tsx
index ccda892..ccf1079 100644
--- a/frontend/src/routes/history/route.tsx
+++ b/frontend/src/routes/history/route.tsx
@@ -14,7 +14,7 @@ const SourcesComponents: React.FC = () => {
   const requests = useSuspenseQuery(requestsQueryOptions);
   const { t } = useTranslation();
 
-  const userHasRequestCreateRole = userHasRoles(user!, ["request_create"]);
+  const userHasRequestCreateRole = userHasRoles(user, ["request_create"]);
 
   return (
     <GenericPanelSearch
diff --git a/frontend/src/routes/sources/$slug.tsx b/frontend/src/routes/sources/$slug.tsx
index ff52297..b107eb7 100644
--- a/frontend/src/routes/sources/$slug.tsx
+++ b/frontend/src/routes/sources/$slug.tsx
@@ -494,7 +494,7 @@ const SecretBadge: React.FC<SourceSecret> = ({ id, name, secretId }) => {
 
 const NotFoundSource: React.FC = () => {
   const { slug } = Route.useParams();
-  return <NotFound title="list.not.found" data={slug} />;
+  return <NotFound title="source.not.found" data={slug} />;
 };
 
 export const Route = createFileRoute("/sources/$slug")({