Skip to content

feat: add centralized logging configuration with structured output (#50) - #73

Open
Sanjeev888 wants to merge 1 commit into
FinVerify:mainfrom
Sanjeev888:feat/logging-config-issue-50
Open

feat: add centralized logging configuration with structured output (#50)#73
Sanjeev888 wants to merge 1 commit into
FinVerify:mainfrom
Sanjeev888:feat/logging-config-issue-50

Conversation

@Sanjeev888

Copy link
Copy Markdown

Closes #50

Summary

Added centralized logging configuration with structured JSON logging support for FinVerify Terminal backend.

Changes

  • Implemented app/logging_config.py with LoggingSettings, JSONFormatter, and setup_logging().
  • Initialized logging at startup in app/main.py.
  • Added unit tests in tests/test_logging_config.py (5 tests passing).
  • Documented LOG_LEVEL and LOG_FORMAT_JSON environment variables in README.md, CONTRIBUTING.md, and .env.example.

Testing

  • pytest tests/test_logging_config.py -> 5 passed
  • Full backend suite (pytest tests) -> 624 passed

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Sanjeev888 is attempting to deploy a commit to the aadi's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Add centralized logging configuration with structured output

1 participant