From f86381e7ad2372fcbc564055e40cac5160d29f22 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 7 Apr 2026 20:38:20 +0000 Subject: [PATCH] fix: use os.IsNotExist for precise check, remove redundant cleanup Agent-Logs-Url: https://github.com/odlev/gin/sessions/2d0f57ad-a46e-45f6-a2f2-b6d4c352f22e Co-authored-by: odlev <65655276+odlev@users.noreply.github.com> --- context.go | 2 +- context_test.go | 3 --- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/context.go b/context.go index d541e50b..8014d4bc 100644 --- a/context.go +++ b/context.go @@ -734,7 +734,7 @@ func (c *Context) SaveUploadedFile(file *multipart.FileHeader, dst string, perm } // Only chmod newly created directories to avoid "operation not permitted" // errors on pre-existing directories we may not own (e.g., /tmp). - if statErr != nil { + if os.IsNotExist(statErr) { if err = os.Chmod(dir, mode); err != nil { return err } diff --git a/context_test.go b/context_test.go index 79d72275..2a3ce70f 100644 --- a/context_test.go +++ b/context_test.go @@ -294,9 +294,6 @@ func TestSaveUploadedFileToExistingDir(t *testing.T) { dir := t.TempDir() dst := filepath.Join(dir, "uploaded.txt") require.NoError(t, c.SaveUploadedFile(f, dst)) - t.Cleanup(func() { - os.Remove(dst) - }) } func TestContextReset(t *testing.T) {