-
Notifications
You must be signed in to change notification settings - Fork 106
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
Rewrite verification algorithm in a way that does not cause layer violations #1377
Comments
I think this is solved so long as there is no language in the core data model about validating proof, or extracting proof from a credential or presentation. |
The issue was discussed in a meeting on 2023-12-13
View the transcript2.6. Rewrite verification algorithm in a way that does not cause layer violations (issue vc-data-model#1377)See github issue vc-data-model#1377. Brent Zundel: rewrite verification algorithm in a way that does not cause layering violations. Manu Sporny: I think the text in the spec is not optimal, and it needs to align with the new interface langauge to align with the securing mechanism, and its done. Orie Steele: Yes, seems like we're in the process of addressing follow-up PRs on verification algorithm. I expect those follow up PRs to resolve this. I'm not sure I have special text to add. Current language isn't acceptable, heading in a good direction. Brent Zundel: Can you make a concrete suggestion that would make the text acceptable? Orie Steele: Yes, I think Mike Jones is planning to do that already? I'll leave it to him. Brent Zundel: can you make a suggestion for text? Manu Sporny: I prefer to explicitly assign mike jones to this. Brent Zundel: Mike is assigned, orie, hopefully you can communicate to mike that he is assigned. |
PR #1393 has started to define an interface that, if merged, will enable this PR to be closed (by utilizing an interface that won't cause layer violations). |
The issue was discussed in a meeting on 2023-12-19
View the transcript2.3. Use new Securing Mechanism Verify Proof interface (pr vc-data-model#1394)See github pull request vc-data-model#1394. Brent Zundel: I'm wondering if it makes sense, Manu, to talk about 1394 before something else. Manu Sporny: I'm fine with any order. There are 3-4 that all build on each other, but they change different things. Brent Zundel: I was seeing the building -- but if the directionality isn't vital we'll just go. See github issue vc-data-model#1377. Brent Zundel: "Use new securing mechanism verify proof interface". This PR has been out for a day. Manu Sporny: This PR is an attempt -- at addressing the same concerns that Mike is trying to address in 1397. The way Jeffrey Yaskin and I are trying to address this is by saying the securing mechanisms must provide an interface that can give you back the secured data. Brent Zundel: Please jump on the queue if you've read this PR. Michael Jones: I will review it, I saw that it existed and skimmed it but decided to finish my PR first. Brent Zundel: Thank you, Mike. Manu Sporny: Mike, when you're reviewing, keep in mind that this isn't an either-or -- it just simplifies a lot of language in your PR from there and we can see what deltas remain. Michael Jones: Makes sense. Brent Zundel: Thank you both. |
The issue was discussed in a meeting on 2023-12-20
View the transcript4.7. Rewrite verification algorithm in a way that does not cause layer violations (issue vc-data-model#1377)See github issue vc-data-model#1377.
Brent Zundel: Issue 1377 a number of PRs 1393, 1394. 1394 has a number of approvals. Though some comments. 1393 not as clear. Already discussed.
|
The issue was discussed in a meeting on 2024-01-03
View the transcript2.1. Rewrite verification algorithm in a way that does not cause layer violations (issue vc-data-model#1377)See github issue vc-data-model#1377. Brent Zundel: there are 4 issues, let's talk about them. first one 1377: Rewrite verification algorithm in a way that does not cause layer violations. looking for a quick status update. all have PR exists. can we successfully address this issue? Manu Sporny: for 1377 believe the vast majority of the changes are already in. Need Mike (selfissued) to look and see what delta is remaining. Believe changes applied through other PRs. Michael Jones: are there more that you believe impact my PR? should I try to resolve merge conflicts and see what happens? Manu Sporny: try to resolve merge conflicts and see what's left. Michael Jones: ack. Ted Thibodeau Jr.: Not just merge conflicts. A number of suggestions have not yet been acted on. Brent Zundel: speaking to a pull request? Ted Thibodeau Jr.: 1397. Manu Sporny: quick heads up to selfissued, there are still some open PRs which impact your PR. just be aware of that. all listed in 1377. Michael Jones: do those look like they're going to land? Manu Sporny: yes. Michael Jones: thanks will look at those first. |
Both @selfissued and @OR13 raised architectural layering concerns around the way that the verification algorithm is written. This issue is being raised to track their concerns.
The text was updated successfully, but these errors were encountered: