Skip to content

Config structure#8

Merged
wistefan merged 12 commits intomainfrom
cert_specific_policies
Apr 28, 2023
Merged

Config structure#8
wistefan merged 12 commits intomainfrom
cert_specific_policies

Conversation

@pulledtim
Copy link
Contributor

No description provided.

@pulledtim pulledtim added the minor Should be applied for new functionality or bigger updates. label Apr 19, 2023
@pulledtim pulledtim self-assigned this Apr 19, 2023
@pulledtim pulledtim force-pushed the cert_specific_policies branch from 5d6cf3c to 58ff098 Compare April 24, 2023 12:12
@pulledtim pulledtim marked this pull request as ready for review April 25, 2023 11:14
@wistefan wistefan merged commit 1e65d83 into main Apr 28, 2023
@wistefan wistefan deleted the cert_specific_policies branch April 28, 2023 11:10
wistefan added a commit that referenced this pull request Mar 5, 2026
## Summary
- Replace trustbloc `verifiable.ParsePresentation` with custom JWT and JSON-LD VP/VC parsing
- New `verifier/jwt_proof_checker.go`: JWT signature verification using DID-resolved keys via lestrrat-go/jwx, handles did:elsi via JAdES
- Delete `verifier/elsi_proof_checker.go` (logic moved to jwt_proof_checker.go)
- JWT VPs/VCs get cryptographic proof verification; JSON-LD VPs parsed without LD-proof verification
- Updated openapi tests: dynamic VP token generation, updated JSON-LD VP test expectations
- SD-JWT parsing still uses trustbloc (Step 8)

## Test plan
- [x] `go build ./...` compiles cleanly
- [x] `go test ./... -count=1` all tests pass
- [x] Signed did:key VP token verification works end-to-end
- [x] did:elsi JAdES verification preserved

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Stefan Wiedemann <wistefan@googlemail.com>
Reviewed-on: http://localhost:3000/wistefan/verifier/pulls/8
Reviewed-by: wistefan <wistefan@googlemail.com>
Co-authored-by: claude <claude@gitea.com>
Co-committed-by: claude <claude@gitea.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Should be applied for new functionality or bigger updates.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants