Skip to content

Conversation

@jrihtarsic
Copy link
Contributor

The purpose of the contributions is to verify (and implement unit tests) for signing and encryption with the EC Brainpool key types.

For details see the PR #293 and SANTUARIO-614

The code is contributed on behalf of the European Commission’s edelivery project to support eDelivery AS4 2.0 profile.

RIHTARSIC Joze added 2 commits April 8, 2024 16:45
add eddsa.jks key creation command to README.txt
Copy link
Contributor

@coheigea coheigea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to generate the encryption keys dynamically rather than using a keystore?

@jrihtarsic
Copy link
Contributor Author

@coheigea The new tests are carried out using the identical "test pattern" employed for other key types". And these tests uses the certificates, not just the keys . Is there a particular reason why the keystore approach is not suitable Certificates using this type of key?

@jrihtarsic
Copy link
Contributor Author

@coheigea, I am not sure if this helps, but I updated the instructions for generating certificates, now "the instruction" is using openssl because java/keytool doesn't support these EC key types anymore.

Copy link
Member

@seanjmullan seanjmullan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. If possible, please fix the typo in the Subject before integrating: s/SANTUIARIO/SANTUARIO/

Also, the subject of SANTUARIO-614 is "edelivery AS4 profile with the brainpool EC keytypes" - I think it should match the title of this PR, which better reflects the change: "Tests for the EC Brainpool key types"

@jrihtarsic jrihtarsic changed the title [SANTUIARIO-614] Tests for the EC Brainpool key types [SANTUARIO-614] Tests for the EC Brainpool key types May 4, 2024
@coheigea coheigea merged commit 6b276fb into apache:main May 7, 2024
coheigea pushed a commit that referenced this pull request May 7, 2024
* [SANTUIARIO-614] Tests for the EC Brainpool key types

* Update XMLSignatureBrainpoolTest with XMLSignatureAbstract
add eddsa.jks key creation command to README.txt

* PR: Update instruction how brainpool.p12 is generated using the openssl + fix typo

---------

Co-authored-by: RIHTARSIC Joze <joze.rihtarsic@ext.ec.europa.eu>
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.

3 participants