-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feature/rqes module #147
base: feature/signatureServices
Are you sure you want to change the base?
Feature/rqes module #147
Conversation
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'm very skeptical because of the dependency between the data modules ... seems to me to be against the purpose of a separate data module
|
||
commonTest { | ||
dependencies { | ||
implementation("at.asitplus.wallet:eupidcredential:${VcLibVersions.eupidcredential}") |
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.
Is this really necessary?
|
||
jvmMain { | ||
dependencies { | ||
implementation(signum.bcpkix.jdk18on) |
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.
Is this really necessary?
} | ||
} | ||
} | ||
//REMOVE ME AFTER REBRANDED ARTIFACT HAS BEEN PUBLISHED |
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.
Remove me!
@@ -32,6 +32,8 @@ kotlin { | |||
commonMain { | |||
dependencies { | |||
api(project(":dif-data-classes")) | |||
api(project(":rqes-data-classes")) |
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.
Doesn't seem to be really needed
A bad attempt at seperating RQES related files into two modules
rqes-data-classes
andvck-rqes
with the latter being a placeholder for additional services which may or may not merge withvck-openid
depending on how pure we want to keep that module.This current version has the following short-comings:
rqes-data-classes
is a dependency for everythingQesInputDescriptor
should be inrqes-data-classes
but is indif-data-classes
for serialization reasonsSignatureRequestParameters
should be inrqes-data-classes
but is inoauth-data-classes
for serialization reasonsBetter ideas/improvements very welcome