Skip to content

Format panic payloads before logging#324

Merged
leynos merged 5 commits intomainfrom
codex/refactor-panic-logging-for-clarity
Aug 19, 2025
Merged

Format panic payloads before logging#324
leynos merged 5 commits intomainfrom
codex/refactor-panic-logging-for-clarity

Conversation

@leynos
Copy link
Copy Markdown
Owner

@leynos leynos commented Aug 18, 2025

Summary

  • format panic payloads before logging to improve clarity
  • update test helpers to format panic payloads
  • cover non-string panic payloads in tests

Testing

  • make fmt
  • make lint
  • make test

closes #218


https://chatgpt.com/codex/tasks/task_e_68a2cd7acb7883228480c0ec8ea02932

Summary by Sourcery

Format panic payloads consistently before logging by converting string payloads to owned strings and using Debug formatting for non-string panics

Enhancements:

  • Update connection task logging to unwrap &str and String panics as owned strings and format all other panic payloads with Debug instead of a placeholder
  • Apply the same panic payload formatting logic in the Wireframe testing helper

Tests:

  • Add a spawn_connection_task test to assert that non-string panic payloads are logged with their Debug representation

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve panic logging by formatting panic payload for clarity

1 participant