fix upload session bugs

Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
butonic · Oct 16, 2024 · 4d34206 · 4d34206
1 parent a322d4a
commit 4d34206
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/changelog/unreleased/fix-upload-session-bugs.md b/changelog/unreleased/fix-upload-session-bugs.md
@@ -0,0 +1,5 @@
+Bugfix: Fix upload session bugs
+
+We fixed an issue that caused a panic when we could not open a file to calculate checksums. Furthermore, we now delete the upload session .lock file on cleanup.
+
+https://github.com/cs3org/reva/pull/
diff --git a/pkg/storage/utils/decomposedfs/upload/session.go b/pkg/storage/utils/decomposedfs/upload/session.go
@@ -86,7 +86,10 @@ func (s *OcisSession) Purge(ctx context.Context) error {
 	if err != nil {
 		return err
 	}
-	defer f.Close()
+	defer func() {
+		f.Close()
+		os.Remove(sessionPath + ".lock")
+	}()
 	if err := os.Remove(sessionPath); err != nil {
 		return err
 	}

diff --git a/pkg/storage/utils/decomposedfs/upload/upload.go b/pkg/storage/utils/decomposedfs/upload/upload.go
@@ -115,7 +115,7 @@ func (session *OcisSession) FinishUpload(ctx context.Context) error {
 
 	sha1h, md5h, adler32h, err := node.CalculateChecksums(ctx, session.binPath())
 	if err != nil {
-		log.Info().Err(err).Msg("error copying checksums")
+		return err
 	}
 
 	// compare if they match the sent checksum