Skip to content

Fix webhook verification and add risk evaluation worker - #454

Merged
Cjay-Cyber-2 merged 2 commits into
ASTROIDX556:mainfrom
samad13:fix/webhook-signatures-risk-worker
Oct 23, 2026
Merged

Cjay-Cyber-2 merged 2 commits into
ASTROIDX556:mainfrom
samad13:fix/webhook-signatures-risk-worker

Conversation

@samad13

@samad13 samad13 commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Align incoming webhook verification with versioned HMAC signatures, preserve legacy signature support, and resolve integration secrets asynchronously.
  • Process risk evaluations through the BullMQ risk-analysis queue, persist retry-safe assessments, and flag transactions that require approval.
  • Confirm the requested composite indexes and API-key authentication guard are already present in the schema/migration history and auth module.

Validation

  • Full test suite: 1,831 tests passed.
  • npm run db:verify passed; database replay was skipped because no shadow database URL was configured.
  • Typecheck and lint passed.

Closes #422
Closes #423
Closes #431
Closes #432

@drips-wave

drips-wave Bot commented Oct 20, 2026

Copy link
Copy Markdown

@samad13 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! 🚀

Learn more about application limits

@Cjay-Cyber-2

Copy link
Copy Markdown
Contributor

The branch now conflicts with current main across transaction/webhook/risk changes. I am holding it rather than guessing at behavior-changing conflict resolution; please rebase the branch onto main, resolve the conflicts, and rerun the green CI checks.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Cjay-Cyber-2
Cjay-Cyber-2 merged commit 80e5286 into ASTROIDX556:main Oct 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants