Skip to content

feat: add VDR capabilities to the cloud-agent using the prism-node#1685

Draft
yshyn-iohk wants to merge 20 commits intomainfrom
feat/vdr-key
Draft

feat: add VDR capabilities to the cloud-agent using the prism-node#1685
yshyn-iohk wants to merge 20 commits intomainfrom
feat/vdr-key

Conversation

@yshyn-iohk
Copy link
Contributor

Description

Summarize the changes you're submitting in a few sentences, including Jira ticket ATL-xxxx if applicable.
Link to any discussion, related issues and bug reports to give the context to help the reviewer understand the PR.

Alternatives Considered (optional)

Link to existing ADR (Architecture Decision Record), if any. If relevant, describe other approaches explored and the selected approach. Documenting why the methods were not selected will create a knowledge base for future reference, helping prevent others from revisiting less optimal ideas.

Checklist

  • My PR follows the contribution guidelines of this project
  • My PR is free of third-party dependencies that don't comply with the Allowlist
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked the PR title to follow the conventional commit specification

@yshyn-iohk yshyn-iohk requested a review from a team as a code owner December 10, 2025 07:20
@yshyn-iohk yshyn-iohk marked this pull request as draft December 10, 2025 07:21
@yshyn-iohk yshyn-iohk changed the title WIP: add VDR capabilities to the cloud-agent using the prism-node feat: add VDR capabilities to the cloud-agent using the prism-node Dec 10, 2025
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
9 Security Hotspots

See analysis details on SonarQube Cloud

@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

Integration Test Results

 23 files   - 1   23 suites   - 1   3s ⏱️ ±0s
 58 tests  - 3   58 ✅  - 3  0 💤 ±0  0 ❌ ±0 
106 runs   - 7  106 ✅  - 7  0 💤 ±0  0 ❌ ±0 

Results for commit 4a09ee6. ± Comparison against base commit ed66a9f.

This pull request removes 4 and adds 1 tests. Note that renamed tests count towards both.
Successfully create and resolve VDR entry using <driver> driver
Successfully deactivate VDR entry using <driver> driver
Successfully share resolvable VDR URL for other party using <driver> driver
Successfully update VDR entry using <driver> driver
Create PRISM DID with internal VDR key

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 10, 2025

Unit Test Results

109 files  + 3  109 suites  +3   21m 41s ⏱️ -6s
912 tests +13  904 ✅ +13  8 💤 ±0  0 ❌ ±0 
919 runs  +13  911 ✅ +13  8 💤 ±0  0 ❌ ±0 

Results for commit c6e44d9. ± Comparison against base commit ed66a9f.

♻️ This comment has been updated with latest results.

@coveralls
Copy link

Coverage Status

coverage: 47.655% (-0.007%) from 47.662%
when pulling dab7ede on feat/vdr-key
into 6d6a65e on main.

Comment on lines 14 to 16
contents: write
packages: write
id-token: write
Copy link
Contributor

@mineme0110 mineme0110 Dec 10, 2025

Choose a reason for hiding this comment

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

move this permissions to job level then global

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
- Fix PrismNodeDIDService to use OperationMaybe from node_api.
- Update integration tests to use the latest snapshot client and GHCR prism-node image.
- Add Gherkin scenario and step to create a PRISM DID with an internal vdr key.
- Document internalKeys.purpose = vdr in DID creation guide.
- switch PRISM_NODE_VERSION to `edge`

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
- Require wallet auth on VDR write endpoints; reads remain public.
- Add optional didKeyId query param to VDR CRUD and thread wallet context through controller/service.
- Update server endpoints/auth wiring and VDR config validation to enforce single driver selection.

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
…r, and REST API with multi-tenancy

- Added prism-node driver config defaults (incl. prismNodeDriverEnabled) and wired VdrService to consume the shared prism-node blocking stub instead of opening its own channel.
- Introduced VdrOperationSigner/PrismNodeVdrOperationSigner to resolve the wallet’s VDR key (multi-tenant), build signed Atala operations, and return MissingVdrKey when absent.
- Updated AppConfig validation/tests and application.conf with the new prism-node config fields; ensured single-driver validation still applies.

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
…yml workflow

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
…key removal and cover with tests

Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
Signed-off-by: Yurii Shynbuiev <yurii.shynbuiev@iohk.io>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 5, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants