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.

Sign up for GitHub

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

Jump to bottom

4 verifier #52

Merged
merged 103 commits into from
Feb 2, 2023
Merged

4 verifier #52

merged 103 commits into from
Feb 2, 2023

Conversation

sgreenbury
Copy link
Collaborator

Completes #4.

sgreenbury and others added 30 commits October 6, 2022 08:24
@sgreenbury
Merge branch '2-resolve-prototype' into 4-verifier
a6e3a3e
@sgreenbury
Initial verifier module outline.
2a1d30a
@sgreenbury
Refactor hashmap use, type alias, fix error variant
412a872
@sgreenbury
Merge branch 'main' into 4-verifier
bf66cd9
@sgreenbury
Add petgraph dep
f7387bf
@sgreenbury
Merge branch '19-controller' into 4-verifier
d014153
@sgreenbury
Outline graph module.
5a5e9b5
@sgreenbury
Add todo for unique nodes
31bb4af
@sgreenbury
Refactor to string, test read
394022d
@thobson88
Merge pull request #43 from alan-turing-institute/19-controller
fd9e9ba 
Merge 19-controller into 4-verifier
@thobson88
Add Chain trait
58f8e8a
Refactor chain did data.
ee0d923
@thobson88
Implement Chain prepend and level
b7c1588
@thobson88
Refactor Chain and Verifier
3a3f8ee
@thobson88
Fix error type in Chain constructor
765bd11
@thobson88
Move root timestamp verification out of Chain
aa87a73
@thobson88
Add methods to Chain trait & start verify_proofs impl
7dda146
@thobson88
Merge branch '29-visualisation' into 4-verifier
b2f6f42
@sgreenbury
Add triple of documents and metadata with proofs and keys
e548ffa
@sgreenbury
get_proof() test and impl
3f3aa17
@sgreenbury
extract_keys tests and impl
4e89842
@sgreenbury
get_controller test and impl
ac860cc
@thobson88
Add DIDChain instance for unit testing
928618d
@thobson88
Add verify_proofs unit test in Chain
4db02ce
@sgreenbury
Move fns from verifier to chain
bad058d
@thobson88
Move impl of Verifier::verify into the trait
50de1d0
@thobson88
Add verifier module to trustchain-ion
e140788
@thobson88
Add verifier module to trustchain-ion lib.rs
ba12675
@sgreenbury
Hashing functions from did-ion
34085a0
@sgreenbury
Incorporate hash in chain, impl verify_proofs
d27a58c
@sgreenbury
Copy link
Collaborator Author

sgreenbury commented Jan 27, 2023 •
edited
Loading

  • Remove the test_chain() function and instead make a str version (and reversed version) importable from data.rs for use both within crate and external to the crate

@sgreenbury
Copy link
Collaborator Author

sgreenbury commented Feb 1, 2023 •
edited
Loading

  • Update Trustchain proof service ID and type with lib consts.

thobson88
thobson88 approved these changes Feb 2, 2023
View reviewed changes
Copy link
Collaborator

@thobson88 thobson88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sgreenbury sgreenbury merged commit c58b388 into main Feb 2, 2023
@sgreenbury sgreenbury deleted the 4-verifier branch February 2, 2023 13:19
This was referenced Feb 2, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thobson88 thobson88 thobson88 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sgreenbury @thobson88