Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Emyrk
Copy link
Member

@Emyrk Emyrk commented Sep 8, 2025

Prior to this change, user and org scopes were always rejected

Prior to this change, user and org scopes were always rejected
@Emyrk Emyrk requested a review from spikecurtis September 8, 2025 16:38
@Emyrk Emyrk changed the title chore: fix rego policy to honor org & user scopes chore: update rego policy to respect user and organisation scopes Sep 8, 2025
Copy link
Contributor

@spikecurtis spikecurtis left a comment

Choose a reason for hiding this comment

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

The org membership thing is unfortunate, but I agree we don't have to solve it in this PR.

@@ -116,7 +123,8 @@ default org := 0
org := org_allow(input.subject.roles)

default scope_org := 0
scope_org := org_allow([input.scope])
scope_org := org_allow([input.subject.scope])

Copy link
Contributor

Choose a reason for hiding this comment

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

nit: extra newline

@Emyrk Emyrk merged commit d527f91 into main Sep 9, 2025
31 checks passed
@Emyrk Emyrk deleted the stevenmasley/org_and_user_scope branch September 9, 2025 10:50
@github-actions github-actions bot locked and limited conversation to collaborators Sep 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants