-
Notifications
You must be signed in to change notification settings - Fork 568
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
Add PK_Signature_Options #4318
Draft
randombit
wants to merge
42
commits into
master
Choose a base branch
from
jack/pk-sign-builder
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Add PK_Signature_Options #4318
Commits on Nov 1, 2024
-
This allows controlling all details of how signatures are created, without having to stuff values into the single parameters string which was previously available.
Configuration menu - View commit details
-
Copy full SHA for 98f9b00 - Browse repository at this point
Copy the full SHA 98f9b00View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2c633fc - Browse repository at this point
Copy the full SHA 2c633fcView commit details -
Configuration menu - View commit details
-
Copy full SHA for aacc762 - Browse repository at this point
Copy the full SHA aacc762View commit details -
Configuration menu - View commit details
-
Copy full SHA for 932f615 - Browse repository at this point
Copy the full SHA 932f615View commit details -
Configuration menu - View commit details
-
Copy full SHA for 35e8dc6 - Browse repository at this point
Copy the full SHA 35e8dc6View commit details -
Configuration menu - View commit details
-
Copy full SHA for 212351f - Browse repository at this point
Copy the full SHA 212351fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3a15aa8 - Browse repository at this point
Copy the full SHA 3a15aa8View commit details -
Configuration menu - View commit details
-
Copy full SHA for ab61a27 - Browse repository at this point
Copy the full SHA ab61a27View commit details -
Configuration menu - View commit details
-
Copy full SHA for c284e9c - Browse repository at this point
Copy the full SHA c284e9cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 88d31d2 - Browse repository at this point
Copy the full SHA 88d31d2View commit details -
Configuration menu - View commit details
-
Copy full SHA for dbf1d07 - Browse repository at this point
Copy the full SHA dbf1d07View commit details -
Configuration menu - View commit details
-
Copy full SHA for 337b924 - Browse repository at this point
Copy the full SHA 337b924View commit details -
Apply suggestions from code review
Co-authored-by: René Meusel <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f26e63e - Browse repository at this point
Copy the full SHA f26e63eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 72e3a81 - Browse repository at this point
Copy the full SHA 72e3a81View commit details -
Make hash optional, add salt len (needed for PSS) Push options type down into EMSA (partially)
Configuration menu - View commit details
-
Copy full SHA for 454b42f - Browse repository at this point
Copy the full SHA 454b42fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 92b5dc5 - Browse repository at this point
Copy the full SHA 92b5dc5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 455f9ef - Browse repository at this point
Copy the full SHA 455f9efView commit details -
Configuration menu - View commit details
-
Copy full SHA for c2512b0 - Browse repository at this point
Copy the full SHA c2512b0View commit details -
Configuration menu - View commit details
-
Copy full SHA for d4e7bce - Browse repository at this point
Copy the full SHA d4e7bceView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0e93767 - Browse repository at this point
Copy the full SHA 0e93767View commit details -
Configuration menu - View commit details
-
Copy full SHA for b025b41 - Browse repository at this point
Copy the full SHA b025b41View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6f75c8e - Browse repository at this point
Copy the full SHA 6f75c8eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5163ff3 - Browse repository at this point
Copy the full SHA 5163ff3View commit details -
Configuration menu - View commit details
-
Copy full SHA for c04b241 - Browse repository at this point
Copy the full SHA c04b241View commit details -
Configuration menu - View commit details
-
Copy full SHA for c1be281 - Browse repository at this point
Copy the full SHA c1be281View commit details -
Configuration menu - View commit details
-
Copy full SHA for 90d854b - Browse repository at this point
Copy the full SHA 90d854bView commit details -
Configuration menu - View commit details
-
Copy full SHA for ddad2c7 - Browse repository at this point
Copy the full SHA ddad2c7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6a71101 - Browse repository at this point
Copy the full SHA 6a71101View commit details -
Configuration menu - View commit details
-
Copy full SHA for a4b80e2 - Browse repository at this point
Copy the full SHA a4b80e2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 88af001 - Browse repository at this point
Copy the full SHA 88af001View commit details -
Configuration menu - View commit details
-
Copy full SHA for efd60ad - Browse repository at this point
Copy the full SHA efd60adView commit details -
Configuration menu - View commit details
-
Copy full SHA for 589524a - Browse repository at this point
Copy the full SHA 589524aView commit details -
Go all-in on the builder pattern
as mentioned here: https://github.com/randombit/botan/pull/4318\#issuecomment-2297990007
Configuration menu - View commit details
-
Copy full SHA for 04fa277 - Browse repository at this point
Copy the full SHA 04fa277View commit details -
Configuration menu - View commit details
-
Copy full SHA for c82da41 - Browse repository at this point
Copy the full SHA c82da41View commit details -
Configuration menu - View commit details
-
Copy full SHA for e07e48c - Browse repository at this point
Copy the full SHA e07e48cView commit details -
PK_Signature_Options::from_legacy()
Without this patch, clang seemed to miscompile the retrofitting of the PK_Signer() legacy constructor. valgrind complained about uninitialized memory when building with clang in -O2 and -O3 (didn't test -O1).
Configuration menu - View commit details
-
Copy full SHA for d2d4e63 - Browse repository at this point
Copy the full SHA d2d4e63View commit details -
Configuration menu - View commit details
-
Copy full SHA for 89a8964 - Browse repository at this point
Copy the full SHA 89a8964View commit details -
Configuration menu - View commit details
-
Copy full SHA for a4ef95a - Browse repository at this point
Copy the full SHA a4ef95aView commit details -
Configuration menu - View commit details
-
Copy full SHA for aa04f2a - Browse repository at this point
Copy the full SHA aa04f2aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9c79af7 - Browse repository at this point
Copy the full SHA 9c79af7View commit details -
Configuration menu - View commit details
-
Copy full SHA for d3dbcc2 - Browse repository at this point
Copy the full SHA d3dbcc2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 9d8cee0 - Browse repository at this point
Copy the full SHA 9d8cee0View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.