revert

config.example.json

@@ -15,8 +15,7 @@
       "label": "Vidéos",
       "dir": "./uploads/videos",
       "allowDirectories": true,
-      "allowedMimeTypes": ["video/*"],
-      "permissions": "0666"
+      "allowedMimeTypes": ["video/*"]
     },
     { "name": "divers", "label": "Divers", "dir": "./uploads/divers" }
   ],

config.schema.json

@@ -48,11 +48,6 @@
                 }
               }
             ]
-          },
-          "permissions":{
-            "type":"string",
-            "default":"0666",
-            "description": "Mode d'écriture sur le disque"
           }
         }
       }

src/config.ts

@@ -60,12 +60,6 @@ export interface DropTarget {
    * valeur globale `allowedMimeTypes`. Toujours résolu après parsing.
    */
   allowedMimeTypes: MimePolicy;
-
-  /**
-   * Mode de permission d'écriture
-   * Par défaut 0o666
-   */
-  permission: number;
 }
 
 /** Liste non vide de cibles de dépôt ; la première est la cible par défaut. */
@@ -391,8 +385,7 @@ function parseTargets(value: unknown, defaultMimePolicy: MimePolicy): DropTarget
       allowedMimeTypes:
         raw.allowedMimeTypes === undefined || raw.allowedMimeTypes === null
           ? defaultMimePolicy
-          : parseMimePolicy(raw.allowedMimeTypes, `targets[${String(index)}].allowedMimeTypes`),
-      permission: parseInt(optionalString(raw.permission, `targets[${String(index)}].permission`) ?? "0666",8)
+          : parseMimePolicy(raw.allowedMimeTypes, `targets[${String(index)}].allowedMimeTypes`)
     };
   };
 
@@ -450,7 +443,5 @@ export function loadConfig(env: NodeJS.ProcessEnv): AppConfig {
     throw new ConfigError(`JSON invalide dans ${file}`, { cause });
   }
 
-  const parsedConfig =  parseConfig(raw);
-  console.log(JSON.stringify(parsedConfig,undefined,2))
-  return parsedConfig;
+  return parseConfig(raw);
 }

src/lib/storage.ts

@@ -33,8 +33,6 @@ export interface SaveFileParams {
   /** Dossier de destination (résolu depuis la cible de dépôt choisie). */
   targetDir: string;
   maxFileSizeBytes: number;
-  /** Permission d'écriture */
-  permission:number;
   /**
    * Mode dossier : recrée `segments` sous `targetDir` et conserve `fileName` ;
    * un fichier existant au même chemin est écrasé. Absent -> nom dérivé unique
@@ -146,7 +144,7 @@ export async function resolveDestination(input: DestinationInput): Promise<Resol
  *   le fichier partiel est supprimé et `PayloadTooLargeError` est levée.
  */
 export async function saveFile(params: SaveFileParams): Promise<StoredFile> {
-  const { source, originalName, mimeType, maxFileSizeBytes,permission } = params;
+  const { source, originalName, mimeType, maxFileSizeBytes } = params;
   const { storedName, destination } = await resolveDestination(params);
 
   let bytesWritten = 0;
@@ -165,7 +163,7 @@ export async function saveFile(params: SaveFileParams): Promise<StoredFile> {
     await pipeline(
       source,
       limiter,
-      createWriteStream(destination, { highWaterMark: WRITE_HIGH_WATER_MARK ,mode:permission}),
+      createWriteStream(destination, { highWaterMark: WRITE_HIGH_WATER_MARK ,mode:0o777}),
     );
   } catch (error) {
     await rm(destination, { force: true });

src/routes/upload.ts

@@ -93,8 +93,7 @@ export const uploadRoutes: FastifyPluginCallback = (app, _options, done) => {
         mimeType: part.mimetype,
         targetDir: target.dir,
         maxFileSizeBytes: app.appConfig.maxFileSizeBytes,
-        ...(placement !== undefined ? { placement } : {}),
-        permission:target.permission
+        ...(placement !== undefined ? { placement } : {})
       });
 
       app.notifier.uploadCompleted({

tests/config.test.ts

@@ -39,8 +39,7 @@ describe('parseConfig', () => {
         label: 'photos',
         dir: './uploads/photos',
         allowDirectories: false,
-        allowedMimeTypes: DEFAULT_ALLOWED_MIME,
-        permission:0o666
+        allowedMimeTypes: DEFAULT_ALLOWED_MIME
       },
     ]);
     expect(config.targets[0]?.allowedMimeTypes).toBe(DEFAULT_ALLOWED_MIME);
@@ -97,8 +96,7 @@ describe('parseConfig', () => {
       label: 'X',
       dir: './x',
       allowDirectories: false,
-      allowedMimeTypes: DEFAULT_ALLOWED_MIME,
-      permission:0o666
+      allowedMimeTypes: DEFAULT_ALLOWED_MIME
     });
     expect(
       parseConfig({ ...BASE, targets: [{ name: 'x', dir: './x', allowDirectories: true }] })

tests/storage.test.ts

@@ -24,8 +24,8 @@ afterEach(async () => {
   await removeDir(uploadDir);
 });
 
-function params(maxFileSizeBytes = 1024 * 1024): { targetDir: string; maxFileSizeBytes: number,permission:number } {
-  return { targetDir: uploadDir, maxFileSizeBytes ,permission:0o666};
+function params(maxFileSizeBytes = 1024 * 1024): { targetDir: string; maxFileSizeBytes: number } {
+  return { targetDir: uploadDir, maxFileSizeBytes };
 }
 
 describe('saveFile', () => {