-
Notifications
You must be signed in to change notification settings - Fork 11
feat: nuvei webhoook integration #448
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
base: main
Are you sure you want to change the base?
Conversation
| use transformers::{get_webhook_object_from_body, NuveiWebhook}; | ||
|
|
||
| let webhook = get_webhook_object_from_body(&request.body) | ||
| .change_context(errors::ConnectorError::WebhookSourceVerificationFailed)?; |
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.
add attach_printable so that error is logged.
|
|
||
| /// Concatenates a vector of strings without any separator | ||
| /// This is useful for creating verification messages for webhooks | ||
| pub fn concat_strings(strings: &[String]) -> String { |
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.
this can be an utill function. So that is needed other connectors can also use it
| } | ||
|
|
||
| /// Helper function to check if client_request_id has payout prefix | ||
| #[allow(dead_code)] |
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.
if it's not being used we can remove the function
Description
nuvei webhook integration
Motivation and Context
Additional Changes
How did you test it?