-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat(Pollux)!: plugins #349
Conversation
a6744c0
to
cfa476a
Compare
ffe2e9c
to
8769062
Compare
8769062
to
047f03d
Compare
@curtis-h The PR is pretty big and I'm going to finish the review soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the changes but not how much testing scenarios we have removed and some validation rules we had. Maybe not restoring anything but I propose to at least improve the test cases that we had in Pollux around the different credentials and what errors to expect
Let's split the work btw
047f03d
to
497047e
Compare
|
497047e
to
e50eae7
Compare
|
e50eae7
to
ad2fe41
Compare
ad2fe41
to
7cb05c2
Compare
Signed-off-by: Curtish <[email protected]>
7cb05c2
to
8fe57af
Compare
Description:
Moving to a plugin architecture.
BREAKING CHANGES:
Pollux completely removed.
Anoncreds not available by default, it needs to be loaded (if wanted).
Agent no longer requires
mediationHandler
andconnectionManager
constructor parameters.removed:
AttachmentFormats
- known Didcomm Message formats, now internal to modules.Message.credentialFormat
- helper function in Message that polluted message logic with Credential logic.Helpers, a large number of helpers, intended for internal use but are technically exported in the Package, so listing below:
Domain.InputLimitDisclosure - specific to modules
Domain.DescriptorItemFormat - specific to modules
Domain.SDJWTVerifiableCredentialProperties - now use
JWT.Claims | SDJWT_VP_PROPS
insteadDomain.JWTVerifiableCredentialProperties - now use
JWT.Claims | JWT_VP_PROPS
insteadDomain.JWTVerifiablePresentationProperties - now use
JWT.Claims | JWT_VC_PROPS | JWT_VP_PROPS
insteadDomain.CredentialSubject
Domain.VerifiableCredentialTypeContainer
Domain.PredicateType
Domain.AttributeType
JWTPresentationClaims
SDJWTPresentationClaims
AnoncredsPresentationClaims
AnoncredsInputFieldFilter
InputFieldFilter
InputField
InputConstraints
InputDescriptor
DefinitionFormat
PresentationAnoncredsRequest
PresentationExchangeDefinitionRequest
PresentationDefinitionData
PresentationDefinitionRequestType
PresentationDefinitionRequest
DescriptorItem
JWTPresentationSubmission
SDJWTPresentationExchangeSubmission
SDJWTPresentationSubmission
AnoncredsPresentationSubmission
PresentationSubmissionData
PresentationSubmission
W3CVerifiableCredentialData
JWTRevocationStatusPurpose
CredentialStatusType
RevocationType
JWTRevocationStatus
JWTProofType
JWTProofPurpose
JWTStatusListResponse
JWTCredentialPayload
JWTPresentationPayload
Hasher
Signer
Verifier
JWTHeader
JWTPayload
PresentationJWTOptions
PresentationRequestOptions
PresentationOptions
AnoncredsPresentationOptions
SDJWPresentationOptions
JWTPresentationOptions
isPresentationDefinitionRequestType
Fixes:
Checklist: