Skip to content
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(router): verify service for applepay merchant registration #2009

Merged
merged 19 commits into from
Aug 31, 2023

Conversation

prajjwalkumar17
Copy link
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

New verify flow for applepay merchant verification

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

@prajjwalkumar17 prajjwalkumar17 added C-feature Category: Feature request or enhancement S-waiting-on-review Status: This PR has been implemented and needs to be reviewed R-waiting-on-L1 Review: Waiting on L1 reviewer A-payments Area: payments labels Aug 24, 2023
@prajjwalkumar17 prajjwalkumar17 added this to the August 2023 Milestone milestone Aug 24, 2023
@prajjwalkumar17 prajjwalkumar17 self-assigned this Aug 24, 2023
@prajjwalkumar17 prajjwalkumar17 requested a review from a team as a code owner August 24, 2023 14:57
crates/api_models/src/verifications.rs Outdated Show resolved Hide resolved
crates/router/src/utils/verification.rs Outdated Show resolved Hide resolved
crates/router/src/lib.rs Outdated Show resolved Hide resolved
jarnura
jarnura previously approved these changes Aug 29, 2023
@jarnura jarnura enabled auto-merge August 29, 2023 10:58
Copy link
Contributor

@Sangamesh26 Sangamesh26 left a comment

Choose a reason for hiding this comment

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

LGTM :)

@jarnura jarnura added this pull request to the merge queue Aug 31, 2023
Merged via the queue into main with commit 636b871 Aug 31, 2023
9 of 10 checks passed
@jarnura jarnura deleted the feat/applepay_merchant_registration_service branch August 31, 2023 07:58
@prajjwalkumar17 prajjwalkumar17 removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed S-ready-for-merge R-waiting-on-L1 Review: Waiting on L1 reviewer labels Aug 31, 2023
Comment on lines +35 to +51
let applepay_internal_merchant_identifier = kms::get_kms_client(kms_config)
.await
.decrypt(encrypted_merchant_identifier)
.await
.change_context(api_error_response::ApiErrorResponse::InternalServerError)?;

let cert_data = kms::get_kms_client(kms_config)
.await
.decrypt(encrypted_cert)
.await
.change_context(api_error_response::ApiErrorResponse::InternalServerError)?;

let key_data = kms::get_kms_client(kms_config)
.await
.decrypt(encrypted_key)
.await
.change_context(api_error_response::ApiErrorResponse::InternalServerError)?;
Copy link
Member

Choose a reason for hiding this comment

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

Can we have this decrypted similar to all the other KMS decryptions that are being done? in kms.rs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-payments Area: payments C-feature Category: Feature request or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants