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

Bbs plus correction and add multi message signing/verifying #18

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

Conversation

rkxx
Copy link

@rkxx rkxx commented Feb 24, 2023

The changes in the pom file are required to run the unit tests (kotlin). Feel free to only merge the source changes and keep the project pure java.

The PrivateKeySigner derived from the ByteSigner only allows signing a single ByteArray. The BBSPlus_PrivateKeySigner extends the PrivateKeySigner by a method for signing a List of ByteArrays. This is required to use this library to create ld-signatures by signing a list of digested statements.

@peacekeeper
Copy link
Member

Hello, in general looks good, but similar to my other comment here decentralized-identity/jsonld-common-java#11 (review), could we not add Kotlin things? Sorry about that :(

@CLAassistant
Copy link

CLAassistant commented Feb 24, 2023

CLA assistant check
All committers have signed the CLA.

@rkxx
Copy link
Author

rkxx commented Feb 27, 2023

Have removed kotlin and kotlin unit tests.

@peacekeeper
Copy link
Member

Thanks, please see the Contributor License Agreement (CLA) above, so we can accept the contribution.

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