-
Notifications
You must be signed in to change notification settings - Fork 74
MLE-28498 11.3.5 Test Fixes for Security Update #1935
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
Changes from all commits
376c4a1
d77b572
864801f
cba38f5
32f21d6
4f463a9
1c0fe2d
056cc83
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,46 @@ | ||||||
| { | ||||||
| "role-name": "java-test-xdmp-login", | ||||||
| "description": "Grants privileges required in ML 11 for invoking server-side transforms and resource extensions", | ||||||
| "privilege": [ | ||||||
| { | ||||||
| "privilege-name": "xdmp-login", | ||||||
| "action": "http://marklogic.com/xdmp/privileges/xdmp-login", | ||||||
| "kind": "execute" | ||||||
| }, | ||||||
| { | ||||||
| "privilege-name": "rest-reader", | ||||||
| "action": "http://marklogic.com/xdmp/privileges/rest-reader", | ||||||
| "kind": "execute" | ||||||
| }, | ||||||
| { | ||||||
| "privilege-name": "xdmp:xslt-eval", | ||||||
| "action": "http://marklogic.com/xdmp/privileges/xslt-eval", | ||||||
|
||||||
| "action": "http://marklogic.com/xdmp/privileges/xslt-eval", | |
| "action": "http://marklogic.com/xdmp/privileges/xdmp-xslt-eval", |
Copilot
AI
May 1, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly, the action URI for xdmp:xslt-invoke should likely match the repo’s convention of prefixing xdmp- in the privilege action (as done for xdmp:invoke, xdmp:eval, etc.). Consider changing it to http://marklogic.com/xdmp/privileges/xdmp-xslt-invoke so the role reliably grants the expected built-in privilege.
| "action": "http://marklogic.com/xdmp/privileges/xslt-invoke", | |
| "action": "http://marklogic.com/xdmp/privileges/xdmp-xslt-invoke", |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,21 +1,41 @@ | ||
| { | ||
| "role-name": "test-rest-writer", | ||
| "description": "Role for test users that can write documents; does not inherit the OOTB rest-writer role so as to avoid having default permissions", | ||
| "privilege": [ | ||
| { | ||
| "privilege-name": "rest-writer", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-writer", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "rest-reader", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-reader", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "rest-tracer", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-tracer", | ||
| "kind": "execute" | ||
| } | ||
| ] | ||
| } | ||
| "role-name": "test-rest-writer", | ||
| "description": "Role for test users that can write documents; does not inherit the OOTB rest-writer role so as to avoid having default permissions", | ||
| "privilege": [ | ||
| { | ||
| "privilege-name": "rest-writer", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-writer", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "rest-reader", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-reader", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "rest-tracer", | ||
| "action": "http://marklogic.com/xdmp/privileges/rest-tracer", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "xdmp-login", | ||
| "action": "http://marklogic.com/xdmp/privileges/xdmp-login", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "xdmp:invoke", | ||
| "action": "http://marklogic.com/xdmp/privileges/xdmp-invoke", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "unprotected-uri", | ||
| "action": "http://marklogic.com/xdmp/privileges/unprotected-uri", | ||
| "kind": "execute" | ||
| }, | ||
| { | ||
| "privilege-name": "unprotected-collections", | ||
| "action": "http://marklogic.com/xdmp/privileges/unprotected-collections", | ||
| "kind": "execute" | ||
| } | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,6 @@ | ||
| { | ||
| "user-name": "rest-admin", | ||
| "description": "rest-admin user", | ||
| "role": [ | ||
| "rest-admin", | ||
| "java-test-delete-temporal" | ||
| ], | ||
| "role": ["rest-admin", "java-test-delete-temporal", "java-test-xdmp-login"], | ||
| "password": "x" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,6 @@ | ||
| { | ||
| "user-name": "rest-evaluator", | ||
| "description": "Test user for the java-client-api-project", | ||
| "role": [ | ||
| "java-test-evaluator" | ||
| ], | ||
| "role": ["java-test-evaluator", "java-test-xdmp-login"], | ||
| "password": "x" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| { | ||
| "user-name": "rest-transform-user", | ||
| "description": "REST API transform execution user", | ||
| "password": "x", | ||
| "role": [ | ||
| "rest-transform-internal", | ||
| "rest-reader", | ||
| "rest-writer", | ||
| "java-test-xdmp-login" | ||
| ], | ||
| "permission": [ | ||
| { "role-name": "rest-reader", "capability": "read" }, | ||
| { "role-name": "rest-writer", "capability": "update" } | ||
| ] | ||
| } | ||
|
jonmille marked this conversation as resolved.
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,6 @@ | ||
| { | ||
| "user-name": "rest-writer", | ||
| "description": "rest-writer user", | ||
| "role": [ | ||
| "rest-writer", | ||
| "java-test-delete-graph" | ||
| ], | ||
| "role": ["rest-writer", "java-test-delete-graph", "java-test-xdmp-login"], | ||
| "password": "x" | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note - these are some old tests that I never got around to modernizing - i.e. they create their own roles instead of depending on ones created via ml-gradle.