From a203b70e272af0bd78653618a1c8f5be764af76a Mon Sep 17 00:00:00 2001
From: Manus <dev-agent@manus.ai>
Date: Sat, 17 Jan 2026 11:12:07 -0500
Subject: [PATCH] =?UTF-8?q?Checkpoint:=20Corrections=20compl=C3=A8tes=20du?=
 =?UTF-8?q?=20syst=C3=A8me=20de=20pi=C3=A8ces=20jointes=20pour=20les=20rap?=
 =?UTF-8?q?pels=20:=20-=20Upload=20de=20fichiers=20avec=20fallback=20autom?=
 =?UTF-8?q?atique=20vers=20stockage=20local=20-=20Configuration=20Nginx=20?=
 =?UTF-8?q?pour=20uploads=20jusqu'=C3=A0=2010=20Mo=20-=20Colonne=20"Pi?=
 =?UTF-8?q?=C3=A8ce=20jointe"=20dans=20le=20tableau=20de=20gestion=20des?=
 =?UTF-8?q?=20rappels=20-=20Support=20des=20fichiers=20locaux=20et=20S3=20?=
 =?UTF-8?q?dans=20l'envoi=20d'emails=20-=20Correction=20de=20l'encoding=20?=
 =?UTF-8?q?base64=20pour=20SMTP/Nodemailer=20-=20Table=20logsrappels=20cr?=
 =?UTF-8?q?=C3=A9=C3=A9e=20dans=20la=20base=20de=20donn=C3=A9es=20MySQL=20?=
 =?UTF-8?q?du=20VPS?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 client/src/pages/AdminRappels.tsx | 12 +++++-
 server/_core/smtpSender.ts        |  1 +
 server/emailService.ts            | 66 ++++++++++++++++++++++++-------
 todo.md                           |  5 +++
 4 files changed, 68 insertions(+), 16 deletions(-)

diff --git a/client/src/pages/AdminRappels.tsx b/client/src/pages/AdminRappels.tsx
index 25d5d3d..fe1bac0 100644
--- a/client/src/pages/AdminRappels.tsx
+++ b/client/src/pages/AdminRappels.tsx
@@ -389,6 +389,7 @@ export default function AdminRappels() {
                   <TableHead>Délai</TableHead>
                   <TableHead>Heure</TableHead>
                   <TableHead>Dates concernées</TableHead>
+                  <TableHead>Pièce jointe</TableHead>
                   <TableHead>Statut</TableHead>
                   <TableHead className="text-right">Actions</TableHead>
                 </TableRow>
@@ -439,6 +440,15 @@ export default function AdminRappels() {
                       <TableCell>
                         {formatDatesInfo(rappel)}
                       </TableCell>
+                      <TableCell>
+                        {rappel.nomFichier && rappel.urlFichier ? (
+                          <Badge variant="outline" className="bg-blue-50 text-blue-700 border-blue-200">
+                            📎 Oui
+                          </Badge>
+                        ) : (
+                          <span className="text-muted-foreground text-sm">Non</span>
+                        )}
+                      </TableCell>
                       <TableCell>
                         <button
                           onClick={() => handleToggleActif(rappel.id, rappel.actif)}
@@ -473,7 +483,7 @@ export default function AdminRappels() {
                   ))
                 ) : (
                   <TableRow>
-                    <TableCell colSpan={7} className="text-center text-muted-foreground">
+                    <TableCell colSpan={8} className="text-center text-muted-foreground">
                       Aucun rappel configuré
                     </TableCell>
                   </TableRow>
diff --git a/server/_core/smtpSender.ts b/server/_core/smtpSender.ts
index 64c0b30..f061ed8 100644
--- a/server/_core/smtpSender.ts
+++ b/server/_core/smtpSender.ts
@@ -59,6 +59,7 @@ export async function sendViaSMTP(params: EmailParams, config: SMTPConfig): Prom
     const attachments = params.attachments?.map(att => ({
       filename: att.filename,
       content: att.content,
+      encoding: 'base64' as const,
       contentType: att.contentType,
     }));
 
diff --git a/server/emailService.ts b/server/emailService.ts
index f46c0dd..8dac7e5 100644
--- a/server/emailService.ts
+++ b/server/emailService.ts
@@ -291,10 +291,22 @@ export async function sendRappelEmail(params: {
   
   if (params.attachmentUrl && params.attachmentFilename && params.attachmentMimeType) {
     try {
-      // Télécharger le fichier depuis S3 et le convertir en base64
-      const response = await fetch(params.attachmentUrl);
-      const buffer = await response.arrayBuffer();
-      const base64 = Buffer.from(buffer).toString('base64');
+      let base64: string;
+      
+      // Vérifier si c'est une URL locale (commence par /uploads/)
+      if (params.attachmentUrl.startsWith('/uploads/')) {
+        // Lire le fichier depuis le système de fichiers
+        const fs = await import('fs/promises');
+        const path = await import('path');
+        const filePath = path.join(process.cwd(), params.attachmentUrl);
+        const fileBuffer = await fs.readFile(filePath);
+        base64 = fileBuffer.toString('base64');
+      } else {
+        // Télécharger le fichier depuis une URL externe (S3)
+        const response = await fetch(params.attachmentUrl);
+        const arrayBuffer = await response.arrayBuffer();
+        base64 = Buffer.from(arrayBuffer).toString('base64');
+      }
       
       attachments.push({
         filename: params.attachmentFilename,
@@ -302,7 +314,7 @@ export async function sendRappelEmail(params: {
         contentType: params.attachmentMimeType,
       });
     } catch (error) {
-      console.error('[Email] Erreur lors du téléchargement de la pièce jointe:', error);
+      console.error('[Email] Erreur lors du chargement de la pièce jointe:', error);
       // Continuer l'envoi sans la pièce jointe
     }
   }
@@ -389,10 +401,22 @@ export async function sendRappelJ7Email(params: {
   
   if (params.attachmentUrl && params.attachmentFilename && params.attachmentMimeType) {
     try {
-      // Télécharger le fichier depuis S3 et le convertir en base64
-      const response = await fetch(params.attachmentUrl);
-      const buffer = await response.arrayBuffer();
-      const base64 = Buffer.from(buffer).toString('base64');
+      let base64: string;
+      
+      // Vérifier si c'est une URL locale (commence par /uploads/)
+      if (params.attachmentUrl.startsWith('/uploads/')) {
+        // Lire le fichier depuis le système de fichiers
+        const fs = await import('fs/promises');
+        const path = await import('path');
+        const filePath = path.join(process.cwd(), params.attachmentUrl);
+        const fileBuffer = await fs.readFile(filePath);
+        base64 = fileBuffer.toString('base64');
+      } else {
+        // Télécharger le fichier depuis une URL externe (S3)
+        const response = await fetch(params.attachmentUrl);
+        const arrayBuffer = await response.arrayBuffer();
+        base64 = Buffer.from(arrayBuffer).toString('base64');
+      }
       
       attachments.push({
         filename: params.attachmentFilename,
@@ -400,7 +424,7 @@ export async function sendRappelJ7Email(params: {
         contentType: params.attachmentMimeType,
       });
     } catch (error) {
-      console.error('[Email] Erreur lors du téléchargement de la pièce jointe:', error);
+      console.error('[Email] Erreur lors du chargement de la pièce jointe:', error);
       // Continuer l'envoi sans la pièce jointe
     }
   }
@@ -490,10 +514,22 @@ export async function sendRappelJ1Email(params: {
   
   if (params.attachmentUrl && params.attachmentFilename && params.attachmentMimeType) {
     try {
-      // Télécharger le fichier depuis S3 et le convertir en base64
-      const response = await fetch(params.attachmentUrl);
-      const buffer = await response.arrayBuffer();
-      const base64 = Buffer.from(buffer).toString('base64');
+      let base64: string;
+      
+      // Vérifier si c'est une URL locale (commence par /uploads/)
+      if (params.attachmentUrl.startsWith('/uploads/')) {
+        // Lire le fichier depuis le système de fichiers
+        const fs = await import('fs/promises');
+        const path = await import('path');
+        const filePath = path.join(process.cwd(), params.attachmentUrl);
+        const fileBuffer = await fs.readFile(filePath);
+        base64 = fileBuffer.toString('base64');
+      } else {
+        // Télécharger le fichier depuis une URL externe (S3)
+        const response = await fetch(params.attachmentUrl);
+        const arrayBuffer = await response.arrayBuffer();
+        base64 = Buffer.from(arrayBuffer).toString('base64');
+      }
       
       attachments.push({
         filename: params.attachmentFilename,
@@ -501,7 +537,7 @@ export async function sendRappelJ1Email(params: {
         contentType: params.attachmentMimeType,
       });
     } catch (error) {
-      console.error('[Email] Erreur lors du téléchargement de la pièce jointe:', error);
+      console.error('[Email] Erreur lors du chargement de la pièce jointe:', error);
       // Continuer l'envoi sans la pièce jointe
     }
   }
diff --git a/todo.md b/todo.md
index b7684c3..3fec7f8 100644
--- a/todo.md
+++ b/todo.md
@@ -848,3 +848,8 @@
 - [x] Corriger le problème du bouton de vidage de l'historique des rappels qui ne fonctionne plus (nom de table incorrect)
 - [ ] Déployer l'intégralité du projet mis à jour sur le VPS (bouton de suppression des rappels et toutes les fonctionnalités manquantes)
 - [x] Configurer l'upload de fichiers pour utiliser le stockage local sur le VPS au lieu du service S3
+- [x] Corriger l'erreur d'upload de fichier depuis le navigateur sur le VPS (augmentation de la limite Nginx à 10 Mo)
+- [x] Ajouter une colonne dans le tableau de gestion des rappels pour indiquer la présence d'une pièce jointe
+- [x] Corriger la détection de la pièce jointe dans le tableau des rappels (vérification de nomFichier et urlFichier au lieu de pieceJointe)
+- [x] Corriger l'envoi de la pièce jointe dans les emails de rappel (support des fichiers locaux et S3)
+- [x] Corriger la corruption de la pièce jointe lors de l'envoi par email (ajout de l'encoding base64 pour SMTP/Nodemailer)