Skip to content

feat(crypto): add RSA-SHA-256 signature verification #9080

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.

Sign up for GitHub

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

Jump to bottom
Merged
likhinbopanna merged 3 commits into from
Sep 3, 2025
Merged

feat(crypto): add RSA-SHA-256 signature verification #9080

likhinbopanna merged 3 commits into from
Sep 3, 2025

Conversation

@Sakilmostak
Copy link
Contributor

@Sakilmostak Sakilmostak commented Aug 28, 2025

Type of Change

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

Description

Add RSA-SHA-256 signature verification in crypto module.

  • rsa crate is used for cryptography implementation

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?

Unit test added
image

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

@Sakilmostak Sakilmostak self-assigned this Aug 28, 2025
@Sakilmostak Sakilmostak requested a review from a team as a code owner August 28, 2025 13:13
@Sakilmostak Sakilmostak added A-core Area: Core flows C-feature Category: Feature request or enhancement labels Aug 28, 2025
@semanticdiff-com
Copy link

semanticdiff-com bot commented Aug 28, 2025 •
edited
Loading

Review changes with  SemanticDiff

Changed Files
File Status
  Cargo.lock Unsupported file format
  crates/common_utils/Cargo.toml Unsupported file format
  crates/common_utils/src/crypto.rs  0% smaller
  crates/hyperswitch_connectors/src/connectors/wise.rs  0% smaller

jarnura
jarnura previously approved these changes Aug 28, 2025
View reviewed changes
ShankarSinghC
ShankarSinghC previously approved these changes Aug 29, 2025
View reviewed changes
@Sakilmostak Sakilmostak dismissed stale reviews from ShankarSinghC and jarnura via 090c878 August 29, 2025 13:57
@Sakilmostak Sakilmostak requested a review from a team as a code owner August 29, 2025 13:57
@likhinbopanna likhinbopanna added this pull request to the merge queue Sep 3, 2025
Merged via the queue into main with commit 3fcbe40 Sep 3, 2025
21 of 25 checks passed
@likhinbopanna likhinbopanna deleted the rsa_sha_256_verification branch September 3, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jarnura jarnura jarnura approved these changes

@ShankarSinghC ShankarSinghC ShankarSinghC approved these changes

Assignees

@Sakilmostak Sakilmostak

Labels

A-core Area: Core flows C-feature Category: Feature request or enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Sakilmostak @jarnura @ShankarSinghC @likhinbopanna