Skip to content

Conversation

@nikhilbonte21
Copy link

No description provided.

@nikhilbonte21 nikhilbonte21 merged commit e46898b into beta Sep 26, 2025
2 of 7 checks passed
"description": "Create/delete process to update datasets for this AIModel",
"resources": [
"entity:default/ai/dataset/*",
"entity:*",
Copy link

Choose a reason for hiding this comment

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

Bug: Policy Pattern Inconsistency Causes Access Issues

Policy resource patterns were inconsistently broadened from specific entities (e.g., default/ai/dataset/*, {entity}) to global wildcards (*). This grants overly broad access for relationship and entity operations. It also creates logical inconsistencies by including both * and {entity}/* patterns within the same policy, making it confusing and redundant.

Fix in Cursor Fix in Web

"description": "Create/delete process to update datasets for this AIModel",
"resources": [
"entity:default/ai/dataset/*",
"entity:*",
Copy link

Choose a reason for hiding this comment

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

Bug: Access Control Policy Too Permissive

The access control policy for entity create/delete operations is overly permissive. The resource changed from entity:default/ai/dataset/* to entity:*, allowing creation/deletion of any entity instead of only AI dataset processes. This grants unintended elevated privileges.

Fix in Cursor Fix in Web

"end-two-entity-type:AIModel",
"end-two-entity-classification:*",
"end-two-entity:{entity}",
"end-two-entity:*",
Copy link

Choose a reason for hiding this comment

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

Bug: Redundant Wildcard Patterns in Policies

Several policy definitions now include both a broad wildcard * and a more specific {entity}/* pattern for the same entity type. This makes the {entity}/* pattern redundant, as the * wildcard already covers all entities, leading to an inconsistent resource specification.

Fix in Cursor Fix in Web

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