Fix for duplicate webhook updates
The retry path can apply the same event twice. The proposed change checks the event ID before applying an update.
- Found
- Repeated delivery reaches the update path without checking whether the event was already handled.
- Proposed change
- Check the event identifier before applying the update; retain the ability to retry failed processing.
- Review focus
- When the event is marked complete, and how that interacts with a failed first attempt.
Checks included
- First delivery
- Duplicate delivery
- Failed-attempt retry
Supporting context
- GitHub issue
- GitHub retry-path files and commit references
- Relevant tests and GitHub check results
Illustrative deliverable, for your team’s review.