fix: webhook retry jitter, search vector refresh, idempotency cleanup… - #1752
Merged
Olowodarey merged 5 commits intoAug 27, 2026
Merged
Conversation
…, signature timestamp - Arena1X#1631: Add jitter to exponential backoff for webhook retries - Arena1X#1632: Refresh search vectors on market entity update - Arena1X#1630: Add TTL cleanup for webhook processed events - Arena1X#1629: Include timestamp in outbound webhook HMAC signatures - Add mimo-related entries to .gitignore
|
@meloball9993-star Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…ill support - Add refreshMarketSearchVector() to SearchService using weighted tsvector - Add backfillMarketSearchVectors() for existing NULL vector rows - MarketsService.update() now auto-refreshes search vector on title/description change - Inject DataSource into SearchService for raw SQL updates - Add tests for refresh and backfill methods
…arch specs MarketsService now depends on SearchService (for search-vector refresh) and SearchService now depends on DataSource (for raw SQL updates), but several spec files that build their own TestingModule providers array weren't updated, so Nest couldn't resolve dependencies at test time.
main gained webhook-dispatcher.service.spec.ts and fuzzy-search.spec.ts via Arena1X#1740 while this branch was in flight. Merging main in surfaces two real conflicts with this PR's own changes: - fuzzy-search.spec.ts builds SearchService without a DataSource mock, which this PR now requires for refreshMarketSearchVector(). - webhook-dispatcher.service.spec.ts asserted retry delays and HMAC signatures assuming no jitter/timestamp, which this PR intentionally adds. Updated the assertions to match the new, intended behavior instead of the old deterministic one.
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
2 participants
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.
Description
This PR fixes critical webhook, search, and backend reliability issues:
Fixes
Webhook Delivery Retry Backoff Schedule (#1631)
Search Vector Refresh on Entity Update (#1632)
Webhook Processed-Events Idempotency (#1630)
Webhook Delivery Signature and Replay Protection (#1629)
All fixes include proper test coverage.
Closes #1631
Closes #1632
Closes #1630
Closes #1629