Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve logging for the Coinbase plugin #1261

Closed
monilpat opened this issue Dec 20, 2024 · 0 comments
Closed

Improve logging for the Coinbase plugin #1261

monilpat opened this issue Dec 20, 2024 · 0 comments
Labels

Comments

@monilpat
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

Currently, our logging for the Coinbase plugin lacks structured formats and essential context, making it challenging to debug issues effectively and trace requests across services.

Describe the solution you'd like

Implement a structured logging framework that:

  • Uses JSON format for all logs.
  • Includes standard fields such as timestamp, severity, and correlation ID.
  • Supports context injection for better observability.
  • Has different log levels (DEBUG, INFO, WARN, ERROR).
  • Allows adding custom fields relevant to the Coinbase plugin operations.

For example, using a logging framework like Winston in Node.js:

const logger = require('winston');
logger.info('User login attempt', { userId: '12345', correlationId: 'abc-123' });

This will greatly assist in correlating events and debugging any issues that may arise.

Describe alternatives you've considered

  • Relying on plain text logs, which can be cumbersome to parse and analyze.
  • Using manual JSON formatting, which is error-prone.
  • Implementing APM tools without a standardized logging approach.

Additional context

Enhancing our logging practices will lead to better monitoring, easier log aggregation, and a more consistent logging pattern, ultimately improving our overall observability and debugging capabilities.

@monilpat monilpat added enhancement New feature or request logging coinbase labels Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant