Skip to content

fix: include metadata in lifecycle logs for deleteObject#1130

Merged
itslenny merged 2 commits into
masterfrom
lenny/fix-delete-metadata
May 28, 2026
Merged

fix: include metadata in lifecycle logs for deleteObject#1130
itslenny merged 2 commits into
masterfrom
lenny/fix-delete-metadata

Conversation

@itslenny
Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Lifecycle logs when deleting a single object do not include metadata. This also impacts wildcard operations via aws cli which internally deletes objects individually

What is the new behavior?

  • Include metadata in lifecycle logs
  • Add webhook missing tests for delete and delete_many
  • Add tests for expected lifecycle logs for all S3 protocol write operations

@itslenny itslenny requested a review from a team as a code owner May 27, 2026 23:29
@itslenny itslenny force-pushed the lenny/fix-delete-metadata branch from d574278 to 8878c54 Compare May 27, 2026 23:33
@coveralls
Copy link
Copy Markdown

coveralls commented May 27, 2026

Coverage Report for CI Build 26575665263

Coverage increased (+0.006%) to 75.265%

Details

  • Coverage increased (+0.006%) from the base build.
  • Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 10654
Covered Lines: 8449
Line Coverage: 79.3%
Relevant Branches: 6241
Covered Branches: 4267
Branch Coverage: 68.37%
Branches in Coverage %: Yes
Coverage Strength: 366.08 hits per line

💛 - Coveralls

@itslenny itslenny merged commit 48c0318 into master May 28, 2026
21 checks passed
@itslenny itslenny deleted the lenny/fix-delete-metadata branch May 28, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants