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

Fix RSA signature verification by hex decoding signatures #5

Merged
merged 1 commit into from
Jun 1, 2024

Conversation

taniwha3
Copy link
Contributor

@taniwha3 taniwha3 commented Jun 1, 2024

Fix RSA signature verification by hex decoding signatures

  • Added hex dependency to Cargo.toml and Cargo.lock for hex decoding
  • Updated src/service/mod.rs to decode hex-encoded signatures before verification

@taniwha3 taniwha3 changed the title hex decode signatures Fix RSA signature verification by hex decoding signatures - Added hex dependency to Cargo.toml and Cargo.lock for hex decoding - Updated src/service/mod.rs to decode hex-encoded signatures before verification - Handle FromHexError for better error reporting when decoding fails This change ensures that signatures are properly hex decoded before attempting to verify them, addressing issues with signature validation. Jun 1, 2024
@taniwha3 taniwha3 changed the title Fix RSA signature verification by hex decoding signatures - Added hex dependency to Cargo.toml and Cargo.lock for hex decoding - Updated src/service/mod.rs to decode hex-encoded signatures before verification - Handle FromHexError for better error reporting when decoding fails This change ensures that signatures are properly hex decoded before attempting to verify them, addressing issues with signature validation. Fix RSA signature verification by hex decoding signatures Jun 1, 2024
    Added hex dependency to Cargo.toml and Cargo.lock for hex decoding
    Updated src/service/mod.rs to decode hex-encoded signatures before verification

Signed-off-by: Tani Aura <[email protected]>
@taniwha3 taniwha3 force-pushed the hex-decode-signatures branch from 584dfee to 893f0d6 Compare June 1, 2024 02:27
@erikreinert erikreinert self-assigned this Jun 1, 2024
@erikreinert erikreinert merged commit 57aaa01 into ALT-F4-LLC:main Jun 1, 2024
2 checks passed
@erikreinert erikreinert added this to the 0.1.0 milestone Jan 3, 2025
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.

2 participants