IUS/once
IUS/once
8
0
Fork 0
Code Issues 6 Pull requests 1 Projects 1 Releases 6 Activity Actions

Log hash/name and reason of removal for removed files #65

Open
heiko wants to merge 2 commits from fix/44-log-removal-details into master
pull from: fix/44-log-removal-details
merge into: IUS:master
Conversation 0 Commits 2 Files changed 2 +9 -7
heiko commented 2026-05-27 17:49:08 +02:00
Owner
Copy link

Log key (UUID), filename, and reason whenever a file is removed.

  • remove.go: log key=<uuid> name="<filename>" by=<user> reason=manual on manual removal; properly handle FindFile errors (return 404)
  • store.go Clean(): promote debug-only messages to log.Printf with structured fields: key, name, reason (info-expired / upload-expired)

Fixes #44

Log key (UUID), filename, and reason whenever a file is removed. - `remove.go`: log `key=<uuid> name="<filename>" by=<user> reason=manual` on manual removal; properly handle FindFile errors (return 404) - `store.go` Clean(): promote debug-only messages to `log.Printf` with structured fields: key, name, reason (info-expired / upload-expired) Fixes #44
- remove.go: log key, filename, user, and reason=manual on manual removal;
  handle FindFile errors properly (return 404)
- store.go Clean(): promote debug messages to log.Printf with structured
  fields: key, name, reason (info-expired / upload-expired)

Fixes #44
heiko requested review from mf 2026-05-27 17:49:50 +02:00 
- remove.go: use %q for by= field (safe if username contains spaces)
- store.go Clean(): add area= field so expired-file logs identify the owner
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/44-log-removal-details:fix/44-log-removal-details
git switch fix/44-log-removal-details

Merge

Merge the changes and update on Forgejo.
git switch master
git merge --no-ff fix/44-log-removal-details
git switch fix/44-log-removal-details
git rebase master
git switch master
git merge --ff-only fix/44-log-removal-details
git switch fix/44-log-removal-details
git rebase master
git switch master
git merge --no-ff fix/44-log-removal-details
git switch master
git merge --squash fix/44-log-removal-details
git switch master
git merge --ff-only fix/44-log-removal-details
git switch master
git merge fix/44-log-removal-details
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
mf
Labels
Clear labels   
nagonag
Automated dependency scan finding

  
nagonag/ignore
Ignore this nagonag finding

  
bug
Something is not working

  
doc
Missing or wrong documentation

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
security
This is security relevant

  
wontfix
This won't be fixed

No labels
nagonag
nagonag/ignore
bug
doc
duplicate
enhancement
help wanted
invalid
question
security
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
IUS/once!65
No description provided.