Skip to content

update IT for add namespace field to policy entry#24

Merged
thjaeckle merged 3 commits intoeclipse-ditto:mainfrom
beyonnex-io:add-ns-field-to-policy
Mar 30, 2026
Merged

update IT for add namespace field to policy entry#24
thjaeckle merged 3 commits intoeclipse-ditto:mainfrom
beyonnex-io:add-ns-field-to-policy

Conversation

@hu-ahmed
Copy link
Copy Markdown
Contributor

No description provided.

@thjaeckle thjaeckle changed the title update IT for add ns field to policy entry update IT for add namespace field to policy entry Mar 23, 2026
@thjaeckle
Copy link
Copy Markdown
Member

@hu-ahmed could you check this branch after the latest changes to the Ditto one?
Test compilation seems to fail, I tried to run the tests on GitHub actions and it failed:

Error:  /home/runner/work/ditto/ditto/ditto-testing/system/src/test/java/org/eclipse/ditto/testing/system/things/rest/ThingsWithImportedPoliciesIT.java:[69,30] error: incompatible types: ImportableType cannot be converted to List<String>

@hu-ahmed hu-ahmed force-pushed the add-ns-field-to-policy branch from 75c2926 to ad1817c Compare March 23, 2026 16:00
@hu-ahmed hu-ahmed force-pushed the add-ns-field-to-policy branch from ad1817c to 3354b92 Compare March 24, 2026 09:18
hu-ahmed and others added 2 commits March 27, 2026 18:46
- Fix parameter order in existing newPolicyEntry 6-param calls
  (correct: label, subjects, resources, namespaces, importable, allowedImportAdditions)
- Add enforcement tests for exact-only and wildcard-only namespace matching
- Add enforcement test verifying absent namespaces means global access
- Add PolicyEntryNamespacesSubResourceIT for GET/PUT /entries/{label}/namespaces
  (CRUD, 404 when never configured, empty array when explicitly set,
   enforcement changes on PUT, global restore on empty PUT)

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Signed-off-by: Thomas Jäckle <[email protected]>
@thjaeckle thjaeckle merged commit 1ff9a83 into eclipse-ditto:main Mar 30, 2026
1 check passed
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.

2 participants