-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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(connector): [INESPAY] Integrate Sepa Bank Debit #6755
base: main
Are you sure you want to change the base?
Conversation
Changed Files
|
707100b
to
eb41804
Compare
890815c
to
ab2aec9
Compare
6ba5f6c
to
e77d529
Compare
let redirection_url = Url::parse(item.response.single_payin_link.as_str()) | ||
.change_context(errors::ConnectorError::FailedToObtainIntegrationUrl)?; | ||
let redirection_data = RedirectForm::from((redirection_url, Method::Get)); | ||
let status = match item.response.status_desc.as_str() { | ||
"Success" => common_enums::AttemptStatus::AuthenticationPending, | ||
_ => common_enums::AttemptStatus::Failure, | ||
}; |
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.
In this case the payment fails, we will throw FailedToObtainIntegrationUrl
error which is not expected. When it is a failed payment, construct a ErrorResponse
, instead TransactionResponse
. Do the same for Psync
Instead of FailedToObtainIntegrationUrl
error, throw a ParsingFailed
error
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.
What are the possible values for item.response.status_desc
according to the connector doc
pub debtor_name: String, | ||
pub debtor_account: 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.
Make these a secret value
let signed_message = hmac::sign(&signing_key, &message); | ||
println!("{:?}<<>>101", signed_message); | ||
let computed_signature = hex::encode(signed_message.as_ref()); | ||
println!("{:?}<<>>102", computed_signature); | ||
println!("{:?}<<>>102", computed_signature.to_lowercase()); | ||
let payload_sign = BASE64_ENGINE.encode(computed_signature); | ||
println!("{:?}<<>>10", payload_sign); | ||
println!("{:?}<<>>11", BASE64_ENGINE.decode(signature.to_owned())); | ||
Ok(payload_sign.as_bytes().eq(&signature)) | ||
} |
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.
Remove all the loggers
223f032
to
9b55b86
Compare
crates/hyperswitch_connectors/src/connectors/inespay/transformers.rs
Outdated
Show resolved
Hide resolved
crates/hyperswitch_connectors/src/connectors/inespay/transformers.rs
Outdated
Show resolved
Hide resolved
9f296e3
to
5139f53
Compare
8f336eb
to
fb56d20
Compare
fad9e70
to
d8e4404
Compare
7cf6ee4
to
caafd29
Compare
caafd29
to
5f20415
Compare
5378fe7
to
4cfa62e
Compare
ab1f93a
to
a795500
Compare
cbf7716
to
326ca2b
Compare
3913410
to
606831b
Compare
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 pr description.
Type of Change
Description
Added payment flows (Auth, PSync, Refund, RSync, and Webhooks) for Inespay Sepa Bank Debit.
Additional Changes
Motivation and Context
How did you test it?
3.Inespay Connector Create:
Response:
Response:
Checklist
cargo +nightly fmt --all
cargo clippy