Skip to content

Conversation

@mwarman
Copy link
Member

@mwarman mwarman commented Dec 17, 2025

📢 Instructions

  • Begin with a DRAFT pull request.
  • Follow italicized instructions to add detail to assist the reviewers.
  • After completing all checklist items, change the pull request to READY.

🔧 Change Summary

Describe the changes included in this pull request. Link to the associated GitHub or Jira issue(s).

  • see Simplify logger #18
  • Updated the logger utility to simplify it. Use pino-lambda formatters for both json and text log formats.
  • Updated the Lambda CDK stack to set the logging threshold to DEBUG to allow the logger utility to control the logging threshold.

📝 Checklist

Pull request authors must complete the following tasks before marking the PR as ready to review.

  • Complete a self-review of changes
  • Unit tests have been created or updated
  • The code is free of [new] lint errors and warnings
  • Update project documentation as needed: README, /docs, JSDoc, etc.

🧪 Steps to Test

Describe the process to test the changes in this pull request.

  1. Deploy to AWS
  2. Regression test all API endpoints, reviewing the logs in CloudWatch.
  3. Try different combinations of logging configuration such as json/text or adjusting the logging level or enabling/disabling.

🔗 Additional Information

Optionally, provide additional details, screenshots, or URLs that may assist the reviewer.

  • [...]

@mwarman mwarman self-assigned this Dec 17, 2025
@mwarman mwarman linked an issue Dec 17, 2025 that may be closed by this pull request
@mwarman mwarman marked this pull request as ready for review December 17, 2025 15:49
@mwarman mwarman merged commit 0136c7c into main Dec 17, 2025
1 check passed
@mwarman mwarman deleted the 18-simplify-logger branch December 17, 2025 16:19
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.

Simplify logger

2 participants