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(rn): add stamping logic to android stamper #1131

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

moldy530
Copy link
Collaborator

@moldy530 moldy530 commented Nov 6, 2024

Pull Request Checklist


PR-Codex overview

This PR focuses on enhancing the NativeTEKStamperModule in the rn-signer project by adding Kotlin serialization support, improving key management, and implementing the stamp function for creating digital signatures.

Detailed summary

  • Added kotlin.plugin.serialization to build.gradle.
  • Introduced ApiStamp data class for serialization.
  • Updated BUNDLE_KEY constants in NativeTEKStamperModule.
  • Implemented the stamp function for signing payloads.
  • Added private key handling with privateKeyToKeyPair.
  • Integrated BouncyCastle as a security provider if not already present.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Nov 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2024 0:20am
aa-sdk-ui-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2024 0:20am

Copy link
Collaborator Author

moldy530 commented Nov 6, 2024

@moldy530 moldy530 marked this pull request as ready for review November 12, 2024 16:22
Base automatically changed from moldy/bundle-injection to moldy/rn-base November 13, 2024 00:27
@moldy530 moldy530 merged commit f3c7adb into moldy/rn-base Nov 13, 2024
5 of 7 checks passed
@moldy530 moldy530 deleted the moldy/rn-stamping branch November 13, 2024 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant