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.

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

Add support for Verifiable Credentials #69

Merged
merged 8 commits into from
Mar 1, 2021
Merged

Conversation

woutslakhorst
Copy link
Member

@woutslakhorst woutslakhorst commented Feb 16, 2021

part of #66

@codecov
Copy link

codecov bot commented Feb 22, 2021

Codecov Report

Merging #69 (d57dba4) into master (a7929ff) will increase coverage by 0.82%.
The diff coverage is 88.07%.

Impacted Files Coverage Δ
vcr/api/v1/api.go 0.00% <0.00%> (ø)
vdr/doc-creator.go 81.48% <ø> (ø)
vcr/vcr.go 70.45% <70.45%> (ø)
vcr/concept/query.go 87.50% <87.50%> (ø)
vcr/concept/template.go 94.00% <94.00%> (ø)
cmd/root.go 85.88% <100.00%> (+0.51%) ⬆️
vcr/concept/registry.go 100.00% <100.00%> (ø)
vcr/concept/types.go 100.00% <100.00%> (ø)
vcr/concept/util.go 100.00% <100.00%> (ø)
vcr/config.go 100.00% <100.00%> (ø)
... and 8 more

woutslakhorst added a commit that referenced this pull request Feb 24, 2021
author Wout Slakhorst <wout.slakhorst@nedap.com> 1613478142 +0100
committer Wout Slakhorst <wout.slakhorst@nedap.com> 1614170222 +0100
gpgsig -----BEGIN PGP SIGNATURE-----

 iQEzBAABCAAdFiEE7J+WX+qKnh5/rHMTXC0I4qbxRDAFAmA2SG4ACgkQXC0I4qbx
 RDD9hQgAkReJ3lsQVNl3SJTwHFNLzK5cg6VYyD+m94Sf/lCNez5q0liE/rtZOznA
 9GGhrf3d3l10RpjKeoTke+n57khXku9aMpGEhEwuNvramilP2NrcidHV0AIP/lPr
 5Lswmy1Uvcc2MLNhc9ftGjcmTXHfrm9DTZFDjIgqQ12MyC0OXHDXsqWka1E0D8fN
 /4RVG5wf3sxcCgy0/q8CwKATDUerCLmHZEZSCSPCunszVv03blOFj/ZI+202HRSH
 DkLCnHm7i33iWUMye1dzZczFu1RIJCOxyOV2Qvw3KfOCLDP6FcF+2/3WZ9L8lGnm
 TeWz8I6hCaqnL7CYT3fJgmE1W7CuVQ==
 =oY0J
 -----END PGP SIGNATURE-----

Added support for Verifiable Credentials:
- concept mapping infrastructure
- instance
- search method on instance
- api spec

this is part of #69

updated nuts libs

fixed generator

go-did update and simplified API return for organization search

add xperimental concept mapping

added example integration for API and network

Updated to latest version of go-did library (#72)

Co-authored-by: Rein Krul <info@reinkrul.nl>

No longer save pubKey in KeyCreator.New (#70)

* No longer directly save a pubKey from in KeyCreator.New

switched go-did back to improve-api branch

something that compiles

connected credential engine

changed pub fields to methods and priv fields

fixed template indexing code

gofmt

some comments

add query tests

completed query.go tests

more template tests

added first test set for credential store

gofmt and goimport

test fix

ignore API for this PR

codeclimate stuff

more interfaces

regen mocks

gofmt:
woutslakhorst added a commit that referenced this pull request Feb 24, 2021
author Wout Slakhorst <wout.slakhorst@nedap.com> 1613478142 +0100
committer Wout Slakhorst <wout.slakhorst@nedap.com> 1614170222 +0100
gpgsig -----BEGIN PGP SIGNATURE-----

 iQEzBAABCAAdFiEE7J+WX+qKnh5/rHMTXC0I4qbxRDAFAmA2SG4ACgkQXC0I4qbx
 RDD9hQgAkReJ3lsQVNl3SJTwHFNLzK5cg6VYyD+m94Sf/lCNez5q0liE/rtZOznA
 9GGhrf3d3l10RpjKeoTke+n57khXku9aMpGEhEwuNvramilP2NrcidHV0AIP/lPr
 5Lswmy1Uvcc2MLNhc9ftGjcmTXHfrm9DTZFDjIgqQ12MyC0OXHDXsqWka1E0D8fN
 /4RVG5wf3sxcCgy0/q8CwKATDUerCLmHZEZSCSPCunszVv03blOFj/ZI+202HRSH
 DkLCnHm7i33iWUMye1dzZczFu1RIJCOxyOV2Qvw3KfOCLDP6FcF+2/3WZ9L8lGnm
 TeWz8I6hCaqnL7CYT3fJgmE1W7CuVQ==
 =oY0J
 -----END PGP SIGNATURE-----

Added support for Verifiable Credentials:
- concept mapping infrastructure
- instance
- search method on instance
- api spec

this is part of #69

updated nuts libs

fixed generator

go-did update and simplified API return for organization search

add xperimental concept mapping

added example integration for API and network

Updated to latest version of go-did library (#72)

Co-authored-by: Rein Krul <info@reinkrul.nl>

No longer save pubKey in KeyCreator.New (#70)

* No longer directly save a pubKey from in KeyCreator.New

switched go-did back to improve-api branch

something that compiles

connected credential engine

changed pub fields to methods and priv fields

fixed template indexing code

gofmt

some comments

add query tests

completed query.go tests

more template tests

added first test set for credential store

gofmt and goimport

test fix

ignore API for this PR

codeclimate stuff

more interfaces

regen mocks

gofmt:
@woutslakhorst woutslakhorst marked this pull request as ready for review February 24, 2021 12:53
@woutslakhorst woutslakhorst changed the title Add support for Verifiable Credentials and NutsCompanyCredential Add support for Verifiable Credentials Feb 24, 2021
credential/concept/query.go Outdated Show resolved Hide resolved
credential/concept/query.go Outdated Show resolved Hide resolved
credential/interface.go Outdated Show resolved Hide resolved
credential/interface.go Outdated Show resolved Hide resolved
credential/interface.go Outdated Show resolved Hide resolved
credential/concept/registry.go Outdated Show resolved Hide resolved
credential/concept/registry.go Outdated Show resolved Hide resolved
credential/concept/registry.go Outdated Show resolved Hide resolved
credential/concept/query.go Outdated Show resolved Hide resolved
credential/concept/query.go Outdated Show resolved Hide resolved
@woutslakhorst
Copy link
Member Author

@reinkrul there are some questions from me on some of the comments

author Wout Slakhorst <wout.slakhorst@nedap.com> 1613478142 +0100
committer Wout Slakhorst <wout.slakhorst@nedap.com> 1614170222 +0100
gpgsig -----BEGIN PGP SIGNATURE-----

 iQEzBAABCAAdFiEE7J+WX+qKnh5/rHMTXC0I4qbxRDAFAmA2SG4ACgkQXC0I4qbx
 RDD9hQgAkReJ3lsQVNl3SJTwHFNLzK5cg6VYyD+m94Sf/lCNez5q0liE/rtZOznA
 9GGhrf3d3l10RpjKeoTke+n57khXku9aMpGEhEwuNvramilP2NrcidHV0AIP/lPr
 5Lswmy1Uvcc2MLNhc9ftGjcmTXHfrm9DTZFDjIgqQ12MyC0OXHDXsqWka1E0D8fN
 /4RVG5wf3sxcCgy0/q8CwKATDUerCLmHZEZSCSPCunszVv03blOFj/ZI+202HRSH
 DkLCnHm7i33iWUMye1dzZczFu1RIJCOxyOV2Qvw3KfOCLDP6FcF+2/3WZ9L8lGnm
 TeWz8I6hCaqnL7CYT3fJgmE1W7CuVQ==
 =oY0J
 -----END PGP SIGNATURE-----

Added support for Verifiable Credentials:
- concept mapping infrastructure
- instance
- search method on instance
- api spec

this is part of #69

updated nuts libs

fixed generator

go-did update and simplified API return for organization search

add xperimental concept mapping

added example integration for API and network

Updated to latest version of go-did library (#72)

Co-authored-by: Rein Krul <info@reinkrul.nl>

No longer save pubKey in KeyCreator.New (#70)

* No longer directly save a pubKey from in KeyCreator.New

switched go-did back to improve-api branch

something that compiles

connected credential engine

changed pub fields to methods and priv fields

fixed template indexing code

gofmt

some comments

add query tests

completed query.go tests

more template tests

added first test set for credential store

gofmt and goimport

test fix

ignore API for this PR

codeclimate stuff

more interfaces

regen mocks

gofmt:
@woutslakhorst
Copy link
Member Author

rebase required force push

vcr/interface.go Outdated Show resolved Hide resolved
vcr/interface.go Outdated Show resolved Hide resolved
vcr/interface.go Outdated Show resolved Hide resolved
vcr/interface.go Show resolved Hide resolved
vcr/interface.go Outdated Show resolved Hide resolved
vcr/interface.go Outdated Show resolved Hide resolved
vcr/test.go Outdated Show resolved Hide resolved
@woutslakhorst woutslakhorst moved this from In Progress to In review in Nuts Reference Implementation v1 Feb 26, 2021
@woutslakhorst woutslakhorst merged commit fae0804 into master Mar 1, 2021
Nuts Reference Implementation v1 automation moved this from In review to Done Mar 1, 2021
@woutslakhorst woutslakhorst deleted the iss66_add_vc branch March 1, 2021 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants