Skip to content

fix: resolve bridge and backend issue regressions - #1579

Merged
Junirezz merged 2 commits into
Junirezz:mainfrom
OtowoSamuel:copilot/resolve-1539-1548-1549-1554
Oct 16, 2026
Merged

Junirezz merged 2 commits into
Junirezz:mainfrom
OtowoSamuel:copilot/resolve-1539-1548-1549-1554

Conversation

@OtowoSamuel

Copy link
Copy Markdown
Contributor

Summary

  • reduce bridge epoch/provider TTL churn
  • align sanitization envelopes with the standard ErrorEnvelope contract
  • validate Deposit/withdraw amount precision and overflow bounds
  • remove plaintext webhook secrets and verify via hash-only comparisons

Changes

  • optimize bridge epoch/provider storage and TTL strategy in contracts/bridge-compat/src/lib.rs
  • standardize sanitization error responses in backend/src/sanitization.ts
  • add Decimal-based money validation and base-unit conversion in backend/src/feeCalculator.ts and backend/src/transferOrchestrator.ts
  • hash-only webhook secret handling in backend/src/webhookDelivery.ts and backend/prisma/schema.prisma
  • backfill migration and targeted regression tests

Closes #1539
Closes #1554
Closes #1549
Closes #1548

@drips-wave

drips-wave Bot commented Oct 16, 2026

Copy link
Copy Markdown

@OtowoSamuel 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

@Junirezz
Junirezz merged commit b0f27b5 into Junirezz:main Oct 16, 2026
12 of 31 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