Merge pull request #98 from Timmoth/dockerfile-permissions-fix

Fixed permissions issue in dockerfile

README.md

@@ -8,7 +8,7 @@ RackPeek is open source and community-driven.
 Code, docs, ideas, bug reports, and real-world usage feedback are all massively appreciated.
 If you run a home lab, you belong here.
 
-[![Join our Discord](https://img.shields.io/badge/Discord-Join%20Us-7289DA?logo=discord&logoColor=white)](https://discord.gg/egXRPdesee) [![Live Demo](https://img.shields.io/badge/Live%20Demo-Try%20RackPeek%20Online-2ea44f?logo=githubpages&logoColor=white)](https://timmoth.github.io/RackPeek/) [![Docker Hub](https://img.shields.io/badge/Docker%20Hub-rackpeek-2496ED?logo=docker&logoColor=white)](https://hub.docker.com/repository/docker/aptacode/rackpeek/general)
+[![Join our Discord](https://img.shields.io/badge/Discord-Join%20Us-7289DA?logo=discord&logoColor=white)](https://discord.gg/egXRPdesee) [![Live Demo](https://img.shields.io/badge/Live%20Demo-Try%20RackPeek%20Online-2ea44f?logo=githubpages&logoColor=white)](https://timmoth.github.io/RackPeek/) [![Docker Hub](https://img.shields.io/badge/Docker%20Hub-rackpeek-2496ED?logo=docker&logoColor=white)](https://hub.docker.com/r/aptacode/rackpeek/)
 
 We’re gathering feedback from homelabbers to validate direction and prioritize features.  
 Answer whichever questions stand out to you, your input directly shapes the project.

RackPeek.Web/Dockerfile

@@ -23,8 +23,10 @@ RUN dotnet publish "./RackPeek.Web.csproj" -c $BUILD_CONFIGURATION -o /app/publi
 FROM base AS final
 WORKDIR /app
 
-# Directory expected to be mounted by the user
+RUN mkdir -p /app/config && chown -R $APP_UID /app/config
+
 VOLUME ["/app/config"]
 
 COPY --from=publish /app/publish .
 ENTRYPOINT ["dotnet", "RackPeek.Web.dll"]
+

RackPeek.Web/Program.cs

@@ -24,30 +24,42 @@ public class Program
         );
 
         var yamlDir = "./config";
+        var yamlFileName = "config.yaml";
+
         var basePath = Directory.GetCurrentDirectory();
 
-        // Resolve yamlDir as relative to basePath
         var yamlPath = Path.IsPathRooted(yamlDir)
             ? yamlDir
             : Path.Combine(basePath, yamlDir);
 
-        if (!Directory.Exists(yamlPath))
-            throw new DirectoryNotFoundException(
-                $"YAML directory not found: {yamlPath}"
-            );
+        Directory.CreateDirectory(yamlPath);
+
+        var yamlFilePath = Path.Combine(yamlPath, yamlFileName);
+
+        if (!File.Exists(yamlFilePath))
+        {
+            // Create empty file safely
+            await using var fs = new FileStream(
+                yamlFilePath,
+                FileMode.CreateNew,
+                FileAccess.Write,
+                FileShare.None);
+            // optionally write default YAML content
+            await using var writer = new StreamWriter(fs);
+            await writer.WriteLineAsync("# default config");
+        }
 
         builder.Services.AddScoped<ITextFileStore, PhysicalTextFileStore>();
 
         var resources = new ResourceCollection();
         builder.Services.AddSingleton(resources);
-        
+
         builder.Services.AddScoped<IResourceCollection>(sp =>
             new YamlResourceCollection(
-                "./config/config.yaml",
+                yamlFilePath,
                 sp.GetRequiredService<ITextFileStore>(),
                 sp.GetRequiredService<ResourceCollection>()));
         
-        
         // Infrastructure
         builder.Services.AddScoped<IHardwareRepository, YamlHardwareRepository>();
         builder.Services.AddScoped<ISystemRepository, YamlSystemRepository>();

notes.md

@@ -28,7 +28,7 @@ chmod +x webui_capture.sh
 docker buildx build \
   --platform linux/amd64,linux/arm64 \
   -f ./Dockerfile \
-  -t aptacode/rackpeek:v0.0.3 \
+  -t aptacode/rackpeek:v0.0.5 \
   -t aptacode/rackpeek:latest \
   --push ..