-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: minting-backend, webhook utility #1732
Conversation
@@ -0,0 +1,3 @@ | |||
# Webhook | |||
|
|||
This package consists of utility to work with [Immutable webhooks](https://docs.immutable.com/docs/zkEVM/products/blockchain-data/webhooks) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO in separate PR: adding more readme.
packages/webhook/sdk/src/init.ts
Outdated
}); | ||
|
||
// check for topic arn prefix | ||
if (!msg.TopicArn.startsWith(allowedTopicArnPrefix[env])) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is there a way to check this first before confirming subscription? Otherwise the partner may confirm a subscription from a malicious actor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aha... right right.
af899f6
packages/minting-backend/sdk/src/minting/dbClient/postgres.ts
Dismissed
Show dismissed
Hide dismissed
packages/minting-backend/sdk/src/minting/dbClient/postgres.ts
Dismissed
Show dismissed
Hide dismissed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approving changes to /blockchain-data, no review of /minting-backend
Hi👋, please prefix this PR's title with:
feat:
,fix:
,refactor:
,docs:
, orchore:
.Summary
For reviewers