stunner icon indicating copy to clipboard operation
stunner copied to clipboard
verified publisher icon l7mp

feat: Add JSON logging support

Open ChaturvediShilpa opened this issue 5 months ago • 2 comments

  • Add --json-log flag and STUNNER_JSON_LOG environment variable
  • Implement slogWriter to redirect standard log output to JSON format
  • Add comprehensive tests for JSON logging functionality
  • Add documentation and example for JSON logging
  • Use Go's slog package for structured JSON logging

This change enables JSON-formatted logging for Stunner, making it easier to integrate with log aggregation systems and monitoring tools.

ChaturvediShilpa avatar Jul 27 '25 10:07 ChaturvediShilpa

Pull Request Test Coverage Report for Build 16550235696

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 73.342%
Totals Coverage Status
Change from base Build 16018826374: 0.0%
Covered Lines: 1095
Relevant Lines: 1493
💛 - Coveralls

coveralls avatar Jul 27 '25 10:07 coveralls

@ChaturvediShilpa It seems pion has just obtained a structured logger: https://github.com/pion/zapion. Seems to miss a couple of functions to be a full drop-in replacement for pion/logging, but it may be useful to track. We would really like to have JSON logging implemented sooner than later.

rg0now avatar Sep 16 '25 15:09 rg0now