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

Remove unnecessary test dependencies on RSA-512 #181

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

coco-speed
Copy link
Collaborator

This pull request was created automatically by CodSpeed to track performance changes of the pull request pyca/cryptography#11444.

The original branch is fork-11444-davidben/test-rsa-512-2

test_unsupported_hash and test_prehashed_digest_mismatch work just fine
with realistic RSA key sizes. (They also, as written, silently test
nothing when the backend rejects RSA-512. As a reminder, RSA-512 was
broken since 1999.)

test_rsa_fips_small_key wants a small key, but I assume RSA-1024 is
fine.
Copy link

codspeed-hq bot commented Aug 17, 2024

CodSpeed Performance Report

Merging #181 will not alter performance

Comparing fork-11444-davidben/test-rsa-512-2 (bbdc241) with main (a542dda)

Summary

✅ 20 untouched benchmarks

@coco-speed coco-speed force-pushed the main branch 20 times, most recently from a0548c1 to 3e9fc8b Compare August 24, 2024 01:05
@coco-speed coco-speed force-pushed the main branch 7 times, most recently from 04d90df to 30167eb Compare August 26, 2024 23:00
@coco-speed coco-speed force-pushed the main branch 27 times, most recently from 94b8b3a to 35a0d9c Compare September 6, 2024 11:05
@coco-speed coco-speed force-pushed the main branch 3 times, most recently from 619c8f2 to ce42b9b Compare September 7, 2024 02:40
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