Copilot commented on code in PR #713:
URL: https://github.com/apache/ranger/pull/713#discussion_r2457835237
##########
agents-common/src/test/resources/policyengine/gds/test_gds_policy_hive_access.json:
##########
@@ -13,135 +13,135 @@
"resource": { "elements": { "database": "sales", "table":
"prospects" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-1" ], "isAllowed": true, "isAudited": true,
"policyId": 2001 }
+ "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"datasetIds": [ 1 ], "allowedByDatasets": [ "dataset-1" ], "isAllowed": true,
"isAudited": true, "policyId": 2001 }
},
{
"name": "column: sales.prospects.channel, user: ds-user, access:
select",
"request": {
"resource": { "elements": { "database": "sales", "table":
"prospects", "column": "channel" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-1" ], "isAllowed": true, "isAudited": true,
"policyId": 2001 }
+ "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"datasetIds": [ 1 ], "allowedByDatasets": [ "dataset-1" ], "isAllowed": true,
"isAudited": true, "policyId": 2001 }
},
{
"name": "table: sales.orders, user: ds-user, access: select",
"request": {
"resource": { "elements": { "database": "sales", "table": "orders" }
},
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-1" ], "isAllowed": true, "isAudited": true,
"policyId": 2001 }
+ "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"datasetIds": [ 1 ], "allowedByDatasets": [ "dataset-1" ], "isAllowed": true,
"isAudited": true, "policyId": 2001 }
},
{
"name": "database: sales, user: ds-user, access: _any",
"request": {
"resource": { "elements": { "database": "sales" } },
"accessType": "", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-1" ], "isAllowed": true, "isAudited": true,
"policyId": 2001 }
+ "result": { "datasets": [ "dataset-1" ], "projects": [ "project-1" ],
"datasetIds": [ 1 ], "allowedByDatasets": [ "dataset-1" ], "isAllowed": true,
"isAudited": true, "policyId": 2001 }
},
{
"name": "table: finance.invoices, user: ds-user, access: select",
"request": {
"resource": { "elements": { "database": "finance", "table":
"invoices" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "allowedByDatasets": [ "dataset-1", "dataset-2" ], "isAllowed":
true, "isAudited": true, "policyId": 2001 }
+ "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "datasetIds": [ 1, 2 ], "allowedByDatasets": [ "dataset-1",
"dataset-2" ], "isAllowed": true, "isAudited": true, "policyId": 2001 }
},
{
"name": "table: finance.invoices, user: ds1-user, access: select",
"request": {
"resource": { "elements": { "database": "finance", "table":
"invoices" } },
"accessType": "select", "user": "ds1-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "allowedByDatasets": [ "dataset-1" ], "isAllowed": true,
"isAudited": true, "policyId": 2001 }
+ "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "datasetIds": [ 1, 2 ], "allowedByDatasets": [ "dataset-1" ],
"isAllowed": true, "isAudited": true, "policyId": 2001 }
},
{
"name": "table: finance.invoices, user: ds2-user, access: select",
"request": {
"resource": { "elements": { "database": "finance", "table":
"invoices" } },
"accessType": "select", "user": "ds2-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "allowedByDatasets": [ "dataset-2" ], "isAllowed": true,
"isAudited": true, "policyId": 2002 }
+ "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "datasetIds": [ 1, 2 ], "allowedByDatasets": [ "dataset-2" ],
"isAllowed": true, "isAudited": true, "policyId": 2002 }
},
{
"name": "table: finance.payments, user: ds-user, access: select",
"request": {
"resource": { "elements": { "database": "finance", "table":
"payments" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "allowedByDatasets": [ "dataset-1", "dataset-2" ], "isAllowed":
true, "isAudited": true, "policyId": 2001 }
+ "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "datasetIds": [ 1, 2 ], "allowedByDatasets": [ "dataset-1",
"dataset-2" ], "isAllowed": true, "isAudited": true, "policyId": 2001 }
},
{
"name": "database: finance, user: ds-user, access: _any",
"request": {
"resource": { "elements": { "database": "finance" } },
"accessType": "", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "allowedByDatasets": [ "dataset-1", "dataset-2" ], "isAllowed":
true, "isAudited": true, "policyId": 2001 }
+ "result": { "datasets": [ "dataset-1", "dataset-2" ], "projects": [
"project-1" ], "datasetIds": [ 1, 2 ], "allowedByDatasets": [ "dataset-1",
"dataset-2" ], "isAllowed": true, "isAudited": true, "policyId": 2001 }
},
{
"name": "table: shipping.shipments, user: ds-user, access: select",
"request": {
"resource": { "elements": { "database": "shipping", "table":
"shipments" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-2" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-2" ], "isAllowed": true, "isAudited": true,
"policyId": 2002 }
+ "result": { "datasets": [ "dataset-2" ], "projects": [ "project-1" ],
"datasetIds": [ 2 ], "allowedByDatasets": [ "dataset-2" ], "isAllowed": true,
"isAudited": true, "policyId": 2002 }
},
{
"name": "database: shipping, user: ds-user, access: _any",
"request": {
"resource": { "elements": { "database": "shipping" } },
"accessType": "", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-2" ], "projects": [ "project-1" ],
"allowedByDatasets": [ "dataset-2" ], "isAllowed": true, "isAudited": true,
"policyId": 2002 }
+ "result": { "datasets": [ "dataset-2" ], "projects": [ "project-1" ],
"datasetIds": [ 2 ], "allowedByDatasets": [ "dataset-2" ], "isAllowed": true,
"isAudited": true, "policyId": 2002 }
},
{
"name": "table: customers.contact_info, user: ds-user, access:
select",
"request": {
"resource": { "elements": { "database": "customers", "table":
"contact_info" } },
"accessType": "select", "user": "ds-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "allowedByDatasets": [ "dataset-3", "dataset-6" ],
"isAllowed": true, "isAudited": true, "policyId": 2003 }
+ "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "datasetIds": [ 3, 6 ], "allowedByDatasets": [
"dataset-3", "dataset-6" ], "isAllowed": true, "isAudited": true, "policyId":
2003 }
},
{
"name": "table: customers.contact_info, user: ds3-user, access:
select",
"request": {
"resource": { "elements": { "database": "customers", "table":
"contact_info" } },
"accessType": "select", "user": "ds3-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "allowedByDatasets": [ "dataset-3" ], "isAllowed":
true, "isAudited": true, "policyId": 2003 }
+ "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "datasetIds": [ 3, 6 ], "allowedByDatasets": [
"dataset-3" ], "isAllowed": true, "isAudited": true, "policyId": 2003 }
},
{
"name": "table: customers.contact_info, user: ds6-user, access:
select",
"request": {
"resource": { "elements": { "database": "customers", "table":
"contact_info" } },
"accessType": "select", "user": "ds6-user", "userGroups": []
},
- "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "allowedByDatasets": [ "dataset-6" ], "isAllowed":
true, "isAudited": true, "policyId": 2006 }
+ "result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "datasetIds": [ 3, 6 ], "datasetIds": [ 3, 6 ],
"allowedByDatasets": [ "dataset-6" ], "isAllowed": true, "isAudited": true,
"policyId": 2006 }
Review Comment:
Duplicate `datasetIds` key in JSON object. The second occurrence should be
removed.
```suggestion
"result": { "datasets": [ "dataset-3", "dataset-6" ], "projects": [
"project-2", "project-4" ], "datasetIds": [ 3, 6 ], "allowedByDatasets": [
"dataset-6" ], "isAllowed": true, "isAudited": true, "policyId": 2006 }
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
