Skip to content

Comments

The logging and monitoring service has been created#8

Open
KAMALDEEN333 wants to merge 2 commits intoMyFanss:mainfrom
KAMALDEEN333:Logging-Setup
Open

The logging and monitoring service has been created#8
KAMALDEEN333 wants to merge 2 commits intoMyFanss:mainfrom
KAMALDEEN333:Logging-Setup

Conversation

@KAMALDEEN333
Copy link

The Logging has been created ensuring it has the below concept and all criterias were met

Integrated a logging service such as Winston, Pino, or extend NestJS's built-in Logger.

Supported for log levels: info, warn, error, debug.

Included context such as timestamps, class/method names, request IDs (where applicable).

Logged formatting: JSON or human-readable based on environment (development vs production).

Captured logs from:

Application lifecycle events (startup, shutdown)
HTTP requests/responses
Errors/exceptions (should integrate with Global Exception Filter)
Custom business events
Monitoring

Integrate lightweight metrics collection (e.g., using Prometheus, OpenTelemetry, or a SaaS tool like Datadog, Sentry, New Relic).

Track metrics such as:

Request count and duration
Error rates
Uptime and memory usage
CPU load (optional)
(Optional) Set up health checks using NestJS's @nestjs/terminus

🧩 Deliverables
A LoggerService or reusable wrapper for logging across the app
Environment-based logging configuration
Metrics/monitoring module setup
Documentation on how to use the logger and interpret the metrics
Example usage in one or more modules (controllers/services)
✅ Acceptance Criteria
Logs are consistent and include meaningful context
Logging can be toggled or filtered via environment config
Monitoring service is integrated and exposes metrics endpoint (e.g., /metrics for Prometheus)
Logging works in harmony with Global Exception Filter
No sensitive data is accidentally logged
All tools and configurations are documented for team use
Activity

@KAMALDEEN333
Copy link
Author

Hi @abdegenius Conflit as been resolved kindly review and merge

@KAMALDEEN333
Copy link
Author

Hi @abdegenius are u available.

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.

1 participant