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.

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

Better Factories #26

Merged
merged 4 commits into from
May 1, 2024
Merged

Better Factories #26

merged 4 commits into from
May 1, 2024

Conversation

paragonie-security
Copy link

  • EccFactory now only returns secure curves by default
    • To get access to insecure curves, pass true as the optional second argument to getNistCurves() and getSecgCurves().
  • Created SecureCurveFactory which only returns Curves and Generators if it's possible to impleemnt securely

@paragonie-security
Better Factories
9b28999 
- EccFactory now only returns secure curves by default
- Created SecureCurveFactory which only returns Curves and Generators if it's possible to impleemnt securely
@paragonie-security
Additional Psalm suppression
6710f95
@paragonie-security
Add unit test to ensure EccFactory doesn't regress
841edde
@paragonie-security
Add Brainpool to factory, fix tests
0a36f11
@paragonie-security paragonie-security merged commit 3760009 into master May 1, 2024
40 checks passed
@paragonie-security paragonie-security deleted the better-curve-factory branch May 1, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@paragonie-security