Skip to content

Conversation

@wolf4ood
Copy link
Contributor

@wolf4ood wolf4ood commented Aug 1, 2025

What this PR changes/adds

adds asset mgmt api v4alpha.

Since it's the first PR after introducing the #5153 it does not contains only asset v4alpha controller but also additional
implementations for:

  • Supporting optional validation on the input object before expansion in JerseyJsonLdInterceptor (used for v4 controllers)
  • For backward compatibility with current controllers, a JerseyJsonLdInterceptor is registered a dynamic resource on each controller of the mgmt API v3
  • Introduced a SchemaType annotation that should be used to annotate the expected schema of an input JsonObject
  • Add missing CatalogAsset type in JSON-LD context and JSON-schema
  • For now the old validation on expanded object is still in place for Asset type. It will get removed once the schema validator is bundled in the management api (currently an add-on extension)

This is marked as breaking-change since it requires to register a JerseyJsonLdInterceptor for custom controllers.

Why it does that

Briefly state why the change was necessary.

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Part of #5137

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@wolf4ood wolf4ood force-pushed the feat/5137_v4aplha_asset_controller branch from 4a2c6ce to 8516997 Compare August 1, 2025 14:35
@wolf4ood wolf4ood added the enhancement New feature or request label Aug 1, 2025
@wolf4ood wolf4ood self-assigned this Aug 1, 2025
@wolf4ood wolf4ood force-pushed the feat/5137_v4aplha_asset_controller branch 5 times, most recently from 50537a5 to 268d807 Compare August 4, 2025 09:58
@github-actions
Copy link

This pull request is stale because it has been open for 7 days with no activity.

@github-actions github-actions bot added the stale Open for x days with no activity label Aug 12, 2025
@wolf4ood wolf4ood removed the stale Open for x days with no activity label Aug 12, 2025
@wolf4ood wolf4ood force-pushed the feat/5137_v4aplha_asset_controller branch from 268d807 to aee7e4d Compare August 18, 2025 08:34
@wolf4ood wolf4ood force-pushed the feat/5137_v4aplha_asset_controller branch from aee7e4d to 3633956 Compare August 18, 2025 12:55
@wolf4ood wolf4ood added the breaking-change Will require manual intervention for version update label Aug 18, 2025
@wolf4ood wolf4ood marked this pull request as ready for review August 18, 2025 13:20
@wolf4ood wolf4ood requested a review from ronjaquensel August 18, 2025 13:20
@wolf4ood wolf4ood merged commit 23d7d8f into eclipse-edc:main Aug 19, 2025
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change Will require manual intervention for version update enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants