From 8a42dddb2ca152c18294147e301faf1856cae37f Mon Sep 17 00:00:00 2001
From: Christoph Hagen <github@christophhagen.de>
Date: Fri, 17 Feb 2023 11:51:11 +0100
Subject: [PATCH] Improve logging a bit

---
 Sources/App/CapServer.swift | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/Sources/App/CapServer.swift b/Sources/App/CapServer.swift
index bc694e4..23adb66 100644
--- a/Sources/App/CapServer.swift
+++ b/Sources/App/CapServer.swift
@@ -279,8 +279,13 @@ final class CapServer {
     // MARK: Counts
 
     private func images(in folder: URL) throws -> [URL] {
-        try fm.contentsOfDirectory(at: folder, includingPropertiesForKeys: nil)
-            .filter { $0.pathExtension == "jpg" }
+        do {
+            return try fm.contentsOfDirectory(at: folder, includingPropertiesForKeys: nil)
+                .filter { $0.pathExtension == "jpg" }
+        } catch {
+            log("Failed to get image urls for cap \(folder.lastPathComponent): \(error)")
+            throw error
+        }
     }
 
     /**
@@ -306,6 +311,7 @@ final class CapServer {
      */
     func save(image data: Data, for cap: Int) throws {
         guard caps[cap] != nil else {
+            log("Tried to save image for unknown cap \(cap)")
             throw CapError.unknownId
         }
         var id = 0
@@ -319,7 +325,12 @@ final class CapServer {
         } else {
             try createImageFolder(for: cap)
         }
-        try data.write(to: capImageUrl)
+        do {
+            try data.write(to: capImageUrl)
+        } catch {
+            log("Failed to write image \(id) for cap \(cap): \(error)")
+            throw CapError.invalidFile
+        }
         caps[cap]!.count = try count(of: cap)
         addChangedImageToLog(cap: cap, image: id)
         log("Added image \(id) for cap \(cap)")