From ac5982f905fd09c21bc114f899157031344ddcb7 Mon Sep 17 00:00:00 2001
From: Pierre-Olivier Mercier <nemunaire@nemunai.re>
Date: Thu, 1 May 2025 13:57:45 +0200
Subject: [PATCH 1/2] fileexporter: Close opened fd

---
 fileexporter/archive.go | 26 +++++++++++++++++++++-----
 fileexporter/copy.go    |  6 +++---
 fileexporter/main.go    | 18 +++++++++++++-----
 3 files changed, 37 insertions(+), 13 deletions(-)

diff --git a/fileexporter/archive.go b/fileexporter/archive.go
index 19544724..d4fc1ceb 100644
--- a/fileexporter/archive.go
+++ b/fileexporter/archive.go
@@ -4,30 +4,46 @@ import (
 	"archive/zip"
 	"errors"
 	"io"
+	"log"
 	"os"
 	"path"
 )
 
 type archiveFileCreator interface {
 	Create(name string) (io.Writer, error)
+	Close() error
+}
+
+type filesCloser []io.Closer
+
+func (fds filesCloser) Close() error {
+	log.Println("Closing fd..")
+	for _, fd := range fds {
+		err := fd.Close()
+		if err != nil {
+			return err
+		}
+	}
+
+	return nil
 }
 
 func init() {
-	OutputFormats["archive"] = func(args ...string) (func(string) (io.WriteCloser, error), error) {
+	OutputFormats["archive"] = func(args ...string) (func(string) (io.WriteCloser, error), io.Closer, error) {
 		if len(args) != 1 {
-			return nil, errors.New("archive has 1 required argument: [destination-file]")
+			return nil, nil, errors.New("archive has 1 required argument: [destination-file]")
 		}
 
 		fd, err := os.Create(args[0])
 		if err != nil {
-			return nil, err
+			return nil, nil, err
 		}
 
 		var w archiveFileCreator
 		if path.Ext(args[0]) == ".zip" {
 			w = zip.NewWriter(fd)
 		} else {
-			return nil, errors.New("destination file has to have .zip extension")
+			return nil, nil, errors.New("destination file has to have .zip extension")
 		}
 
 		return func(dest string) (io.WriteCloser, error) {
@@ -37,6 +53,6 @@ func init() {
 			}
 
 			return NopCloser(fw), nil
-		}, nil
+		}, filesCloser{w, fd}, nil
 	}
 }
diff --git a/fileexporter/copy.go b/fileexporter/copy.go
index 6610b2c1..d4c7b96a 100644
--- a/fileexporter/copy.go
+++ b/fileexporter/copy.go
@@ -8,15 +8,15 @@ import (
 )
 
 func init() {
-	OutputFormats["copy"] = func(args ...string) (func(string) (io.WriteCloser, error), error) {
+	OutputFormats["copy"] = func(args ...string) (func(string) (io.WriteCloser, error), io.Closer, error) {
 		if len(args) > 1 {
-			return nil, errors.New("copy can only take 1 argument: [destination-folder]")
+			return nil, nil, errors.New("copy can only take 1 argument: [destination-folder]")
 		}
 
 		if len(args) == 1 {
 			fic.FilesDir = args[0]
 		}
 
-		return nil, nil
+		return nil, nil, nil
 	}
 }
diff --git a/fileexporter/main.go b/fileexporter/main.go
index 2fa95ab9..5fea7f6d 100644
--- a/fileexporter/main.go
+++ b/fileexporter/main.go
@@ -14,7 +14,7 @@ import (
 	"srs.epita.fr/fic-server/libfic"
 )
 
-var OutputFormats = map[string]func(...string) (func(string) (io.WriteCloser, error), error){}
+var OutputFormats = map[string]func(...string) (func(string) (io.WriteCloser, error), io.Closer, error){}
 
 func exportThemeFiles(tdir string) (errs error) {
 	theme, exceptions, err := sync.BuildTheme(sync.GlobalImporter, tdir)
@@ -129,6 +129,13 @@ func main() {
 
 	log.Println("Using", sync.GlobalImporter.Kind())
 
+	hasError := doExport()
+	if hasError {
+		os.Exit(1)
+	}
+}
+
+func doExport() bool {
 	// Configure destination
 	if flag.NArg() < 1 {
 		var formats []string
@@ -147,7 +154,10 @@ func main() {
 
 		log.Fatal("Please define wanted output format between [" + strings.Join(formats, " ") + "]")
 	} else {
-		fw, err := outputFormat(flag.Args()[1:]...)
+		fw, closer, err := outputFormat(flag.Args()[1:]...)
+		if closer != nil {
+			defer closer.Close()
+		}
 		if err != nil {
 			log.Fatal(err)
 		} else if fw != nil {
@@ -170,7 +180,5 @@ func main() {
 		}
 	}
 
-	if hasError {
-		os.Exit(1)
-	}
+	return hasError
 }

From 0c619583a323e6f3bd7926b9d53ab4c9b25ac0d8 Mon Sep 17 00:00:00 2001
From: Renovate Bot <renovate@pomail.fr>
Date: Thu, 8 May 2025 09:48:09 +0000
Subject: [PATCH 2/2] chore(deps): update dependency sass to v1.87.0

---
 frontend/fic/package-lock.json | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/frontend/fic/package-lock.json b/frontend/fic/package-lock.json
index d08b942c..eabdfd8f 100644
--- a/frontend/fic/package-lock.json
+++ b/frontend/fic/package-lock.json
@@ -2881,9 +2881,9 @@
       }
     },
     "node_modules/sass": {
-      "version": "1.86.0",
-      "resolved": "https://registry.npmjs.org/sass/-/sass-1.86.0.tgz",
-      "integrity": "sha512-zV8vGUld/+mP4KbMLJMX7TyGCuUp7hnkOScgCMsWuHtns8CWBoz+vmEhoGMXsaJrbUP8gj+F1dLvVe79sK8UdA==",
+      "version": "1.87.0",
+      "resolved": "https://registry.npmjs.org/sass/-/sass-1.87.0.tgz",
+      "integrity": "sha512-d0NoFH4v6SjEK7BoX810Jsrhj7IQSYHAHLi/iSpgqKc7LaIDshFRlSg5LOymf9FqQhxEHs2W5ZQXlvy0KD45Uw==",
       "dev": true,
       "license": "MIT",
       "dependencies": {