From e51da7f3596bdb81da66fa01a8bcad9b63450cec Mon Sep 17 00:00:00 2001
From: Paul Schneider <paulschneider@free.fr>
Date: Tue, 18 May 2021 00:39:03 +0100
Subject: [PATCH] TODO policy on existing zip from fs

---
 .../Controllers/PackagesController.cs         | 44 ++++++++++++++++---
 1 file changed, 39 insertions(+), 5 deletions(-)

diff --git a/src/nuget-host/Controllers/PackagesController.cs b/src/nuget-host/Controllers/PackagesController.cs
index d4580af..83648f2 100644
--- a/src/nuget-host/Controllers/PackagesController.cs
+++ b/src/nuget-host/Controllers/PackagesController.cs
@@ -54,7 +54,10 @@ namespace nuget_host.Controllers
             var clearkey = protector.Unprotect(apiKey);
             var apikey = dbContext.ApiKeys.SingleOrDefault(k => k.Id == clearkey);
             if (apikey == null)
+            {
+                logger.LogInformation("403 : no api-key");
                 return Unauthorized();
+            }
 
             foreach (var file in Request.Form.Files)
             {
@@ -94,7 +97,11 @@ namespace nuget_host.Controllers
                                 var dest = new FileInfo(path);
                                 var destdir = new DirectoryInfo(dest.DirectoryName);
                                 if (dest.Exists)
-                                    return BadRequest(new { error = "existant" });
+                                {
+                                    ViewData["error"] = "existe déjà";
+                                    logger.LogInformation("400 : existe déjà");
+                                    return BadRequest(ViewData);
+                                }
 
                                 if (!destdir.Exists)
                                 {
@@ -128,12 +135,39 @@ namespace nuget_host.Controllers
                                     if (pkg == null)
                                     {
                                         // TODO Choose an app policy to take ownership
-                                        // on existing package on disk. 
-                                        throw new Exception("Package directory exists, but don't have any owner");
+                                        // and populate db with zip info
+                                        // from either existing package on disk,
+                                        // or from request
+                                        logger.LogError("Package directory exists, but don't have any owner");
+                                        throw new NotImplementedException();
+
+                                        Package newpkgfromdisk = new Package{
+                                            Id = pkgid,
+                                            Description = pkgdesc,
+                                            OwnerId = apikey.UserId
+                                        };
+                                        dbContext.Packages.Add(newpkgfromdisk);
+                                        var newversionfromdisk = new PackageVersion
+                                        {
+                                            Package = newpkgfromdisk,
+                                            Major = version.Major,
+                                            Minor = version.Minor,
+                                            Patch = version.Patch,
+                                            IsPrerelease = version.IsPrerelease,
+                                            FullString = version.ToFullString()
+                                        };
+                                        dbContext.PackageVersions.Add(newversionfromdisk);
+                                        await dbContext.SaveChangesAsync();
+                                        logger.LogInformation($"new package : {entry.Name}");
+                                    
+
                                     }
 
                                     if (apikey.UserId != pkg.OwnerId)
+                                    {
+                                        logger.LogInformation("403 : not owner");
                                         return Unauthorized();
+                                    }
 
                                     var newversion = new PackageVersion
                                     {
@@ -155,9 +189,9 @@ namespace nuget_host.Controllers
                 }
                 catch (Exception ex)
                 {
-                    logger.LogError($"400: {file.Name}");
+                    logger.LogError($"400 en traitant {file.Name}:");
                     logger.LogError(ex.Message);
-                    return new BadRequestObjectResult(ViewData);
+                    throw;
                 }
             }
             return Ok(ViewData);