Do not log during expiration if the versionID

Seragonia · Seragonia · commit dc4da18c6557 · 2025-03-05T16:33:03.000+01:00
Issue: CLDSRV-613
diff --git a/lib/api/objectDelete.js b/lib/api/objectDelete.js
@@ -89,7 +89,8 @@ function objectDeleteInternal(authInfo, request, log, isExpiration, cb) {
                 }
 
                 if (versioningCfg && versioningCfg.Status === 'Enabled' &&
-                    objMD.versionId === reqVersionId && isExpiration) {
+                    objMD.versionId === reqVersionId && isExpiration &&
+                    !objMD.isDeleteMarker) {
                         log.warn('expiration is trying to delete a master version ' +
                         'of an object with versioning enabled', {
                             method: 'objectDeleteInternal',

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,8 @@ function objectDeleteInternal(authInfo, request, log, isExpiration, cb) {`
`89`	`89`	`}`
`90`	`90`
`91`	`91`	`if (versioningCfg && versioningCfg.Status === 'Enabled' &&`
`92`		`- objMD.versionId === reqVersionId && isExpiration) {`
	`92`	`+ objMD.versionId === reqVersionId && isExpiration &&`
	`93`	`+ !objMD.isDeleteMarker) {`
`93`	`94`	`log.warn('expiration is trying to delete a master version ' +`
`94`	`95`	`'of an object with versioning enabled', {`
`95`	`96`	`method: 'objectDeleteInternal',`