Skip to content

fix: log document content at debug level#922

Open
mrab54 wants to merge 1 commit into
icereed:mainfrom
mrab54:fix/921-log-document-content-at-debug-level
Open

fix: log document content at debug level#922
mrab54 wants to merge 1 commit into
icereed:mainfrom
mrab54:fix/921-log-document-content-at-debug-level

Conversation

@mrab54

@mrab54 mrab54 commented Mar 6, 2026

Copy link
Copy Markdown

Summary

Summary by CodeRabbit

Release Notes

No user-visible changes in this release. Internal logging adjustments have been made to improve development diagnostics without affecting application functionality.

@coderabbitai

coderabbitai Bot commented Mar 6, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Modified logging behavior in the UpdateDocuments function to emit content field updates at debug level instead of info level, reducing verbose logging of full document contents during OCR operations.

Changes

Cohort / File(s) Summary
Logging Level Adjustment
paperless.go
Changed logging for "content" field updates from info level (Printf) to debug level (Debugf) in the UpdateDocuments update loop. All other fields continue logging at info level.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A whispered fix in quiet code,
Content secrets now light their load,
At debug we tuck them away so still,
While other fields log on as they will.
Shh... the OCR logs with grace!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'fix: log document content at debug level' directly and accurately summarizes the main change: converting document content logging from info to debug level.
Linked Issues check ✅ Passed The PR addresses issue #921 by changing document content logging from info level to debug level, directly meeting the stated objective to reduce log verbosity.
Out of Scope Changes check ✅ Passed The changes are limited to logging behavior for content field updates; no unrelated or out-of-scope modifications are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

Document content values logged at info level

1 participant