Add push notification trace UUID tracking for debugging#397
Open
Add push notification trace UUID tracking for debugging#397
Conversation
Extract push_trace_uuid from the Pushmobile UserEvent, include it in the notification payload, and log it at each step (bus_consume, service, notifier) for end-to-end push notification tracing. https://claude.ai/code/session_0193K1vomcUqeAw3TAif3MCK
|
Contributor
|
Build succeeded. ✔️ tox-linters SUCCESS in 3m 03s |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Summary
This change adds support for tracking push notifications through a trace UUID, enabling better debugging and tracing of push notification flows across the system.
Key Changes
Added
push_trace_uuidparameter to the push notification workflow:send_push_notification()method with default empty stringWAZO_PUSH_TRACE_UUIDfieldEnhanced logging throughout the push notification lifecycle:
services.pyservices.pynotifier.py(both send and cancel operations)Updated test coverage:
WAZO_PUSH_TRACE_UUIDto test event dataImplementation Details
https://claude.ai/code/session_0193K1vomcUqeAw3TAif3MCK
Note
Low Risk
Low risk: changes are additive (optional
push_trace_uuidfield) and primarily affect logging and bus payload metadata for push notifications/cancellation.Overview
Adds end-to-end propagation of an optional
push_trace_uuidfor mobile push notifications, extracted fromUserEvent(WAZO_PUSH_TRACE_UUID) and included in the payload sent on the bus.Improves observability by logging the trace UUID when receiving, sending, publishing, and cancelling push notifications, and updates the
test_bus_consumeassertion to verify the new argument is passed through.Written by Cursor Bugbot for commit 56489ee. This will update automatically on new commits. Configure here.