#815: fix(logging): remove duplicate logger initialization in images utils

[kri-jha]

Fix duplicate logger initialization

Problem
The images utility initialized the logger twice, overwriting the project’s
custom logging configuration.

Solution

• Removed duplicate logging.getLogger() usage
• Removed unused logging import
• Kept only get_logger(__name__)

Result
Ensures consistent logging behavior and improves code quality.

Closes #815

Summary by CodeRabbit

• Chores
  • Removed redundant logging setup to streamline internal code structure and improve maintainability without affecting user-facing functionality.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Removed duplicate logger initialization and unused logging import from backend/app/utils/images.py. The file now relies solely on the project's custom get_logger() function for consistent, properly configured logging behavior.

Changes

Cohort / File(s)	Change Summary
Bug Fix: Duplicate Logger Cleanup backend/app/utils/images.py	Removed unused import logging statement and redundant logger initialization that was overwriting the custom get_logger() setup. Consolidates to a single logger instance using the project's centralized logging system.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Straightforward removal of duplicate code with no functional impact
• Single file affected with clear, minimal changes
• No complex logic or dependencies to trace

Poem

🐰 Two loggers quarreled in the code,
One custom, one standard—quite the load!
Now unified, the conflict's gone,
Color-coded logs march proudly on! ✨

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: removing duplicate logger initialization in images.py utilities, which is the core objective of this PR.
Linked Issues check	✅ Passed	The PR fulfills all coding requirements from issue #815: removes duplicate logger initialization on line 29, removes unused logging import, and retains the project's get_logger(name) approach for consistent logging configuration.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to the requirements in issue #815 - only the images.py file was modified to remove duplicate logger initialization and the unused logging import.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d07d817 and a9e9459.

📒 Files selected for processing (1)

• backend/app/utils/images.py

💤 Files with no reviewable changes (1)

• backend/app/utils/images.py

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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