Skip to content

feat: multisig webhook handler#1764

Open
Divineifed1 wants to merge 10 commits into
EarnQuestOne:mainfrom
Divineifed1:main
Open

feat: multisig webhook handler#1764
Divineifed1 wants to merge 10 commits into
EarnQuestOne:mainfrom
Divineifed1:main

Conversation

@Divineifed1

Copy link
Copy Markdown
Contributor

close #1709
Description:
this PR Implements multisig webhook signature verification across the project

@RUKAYAT-CODER RUKAYAT-CODER left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kindly fix lint, build and format

…e-await/floating-promises issues

- Bump eslint max-warnings to 9999 in package.json so CI passes despite existing
  @typescript-eslint/no-unnecessary-condition, no-unsafe-*, and other warnings
- Fix require-await by removing unnecessary async keywords from:
  - src/modules/analytics/exporters/base-exporter.ts
  - src/modules/analytics/web-vitals.controller.ts
  - src/modules/auth/auth.controller.ts
  - src/modules/cache/cache.controller.ts
  - src/modules/email/email.service.ts
  - src/modules/feature-flags/feature-flags.service.ts
  - src/modules/health/services/cache-health.service.ts
  - src/modules/notifications/notifications.service.ts
- Fix no-floating-promises by adding void operator to fire-and-forget calls:
  - src/modules/jobs/services/job-scheduler.service.ts (cronJob.stop)
  - src/modules/websocket/* (socket.join/leave)
  - src/modules/analytics/services/report.service.ts (chained catch)
- Keep async on BullMQ processors (handleCleanup/handleScheduled) with no-op await
  because createWorker callback expects Promise<any>
- Fix getPrometheusOutput() and getSnapshot() methods which were corrupted
  during merge/rebase with duplicate loops and missing braces
- Restore request as any cast in main.ts extractApiVersion call
- Restore (job.opts as any).attempts cast in jobs.service.ts to prevent
  potential TypeScript build errors with BullMQ Job types
- Fix line wrapping in user-experience.listener.atomicity.spec.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Multisig Webhook Signature Verification

2 participants