-
Notifications
You must be signed in to change notification settings - Fork 9
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
Merging spike to main [WIP] #4
Conversation
- Refactors tests by removing duplicate code. - Adds EBSI test vectors - Adds comments on test methods. - Adds placeholders to some simple test plans. These changes are made towards adding adding simple DID creation and VC signing flows and creating VC presentations.
@veikkoeeva Following the conversation I am doing the merge and this some of the thoughts of the following actions, i will create an issue with the following items todo and discuss. The points of debate and issues with different implementations has been around the different libraries required and dependencies, so most of the work is to decouple this the implementations. This spike contains the most uptodate model so it will be easier to move to the main project. Steps:
Edit: And lots more.. :) |
Sounds like a plan. I agree on decoupling the cryptographic provider and removing attributes from the data model. It is maybe one example on how to wrap Bouncy to Azure specific APIs, or use Azure specific APIs in general (those reading this, some links in code to docs how to extend them). would prefer to have working tests and some crypto API to work with fairly soon to move ahead. I don't mind if it gets completely swapped but if there's isn't limbo for too long. I will record a few thoughts I had about crypto in Monday or Tuesday likely if you don't already start one. I do not know if there is a need to record why converters indead of, say, "a bag" on having a wrapper class to access attributes (my personal reasons, amonst some others: working with types looks easier and we can reference directly those to spec as links in code show, extendable without refactoring core -- while acknowledging they should be further refactored). Maybe record if we want to maintain a constraint that the code is useable in Blazor, Apple devices and so on. The spike uses attributes to jsonify .NET naming but it can also be configured, I don't about extension data. I record this tersely like this if someone reads this and thinks we can work without any attributes, could be possible. We need builders and some other APIs too. |
@veikkoeeva nice I am trying to move your tests now for the convertors so that part is functional and we can think of a common interface to deserialise. |
@veikkoeeva well all the tests are failing due to this dotnet/runtime#29975 mainly DataMember does not work in system.text.json i truly don't believe it.. and is not coming until 6 or further down the line.. so not netstandard support either. |
DO NOT MERGE YET
This is currently a rebase onto main to create a common history when there was previously none. Needs to be refactored so the code fits into the current structure and vice versa (and likely squashed with a reasonable commit message).