Skip to content

Feature/enhance logger service circular bug#122

Merged
mosufy merged 2 commits intoreflex-media:developfrom
daison12006013:feature/enhance-logger-service-circular-bug
Nov 13, 2025
Merged

Feature/enhance logger service circular bug#122
mosufy merged 2 commits intoreflex-media:developfrom
daison12006013:feature/enhance-logger-service-circular-bug

Conversation

@daison12006013
Copy link
Copy Markdown
Collaborator

@daison12006013 daison12006013 commented Nov 13, 2025

  • Added sanitizeForJson method to handle functions, circular references, and other data types for safe JSON serialization.
  • Updated logging methods to use sanitized messages, ensuring consistent output.
  • Expanded test cases to verify handling of functions, circular references, and complex data structures in logger extras.

mosufy
mosufy previously approved these changes Nov 13, 2025
@mosufy mosufy changed the base branch from master to develop November 13, 2025 07:24
@mosufy mosufy dismissed their stale review November 13, 2025 07:24

The base branch was changed.

Daison Carino added 2 commits November 13, 2025 15:26
- Added `sanitizeForJson` method to handle functions, circular references, and other data types for safe JSON serialization.
- Updated logging methods to use sanitized messages, ensuring consistent output.
- Expanded test cases to verify handling of functions, circular references, and complex data structures in logger extras.
- Added `sanitizeForJson` method to handle functions, circular references, and other data types for safe JSON serialization.
- Updated logging methods to utilize the new sanitization, ensuring consistent and safe output for complex objects.
@mosufy mosufy force-pushed the feature/enhance-logger-service-circular-bug branch from a56d097 to 6f77bfe Compare November 13, 2025 07:26
@mosufy mosufy merged commit 945f8c1 into reflex-media:develop Nov 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants