Skip to content

feat: organization tuples as contextual checks#462

Merged
golanglemonade merged 4 commits into
mainfrom
feat-parent-context-tuple
May 18, 2026
Merged

feat: organization tuples as contextual checks#462
golanglemonade merged 4 commits into
mainfrom
feat-parent-context-tuple

Conversation

@golanglemonade
Copy link
Copy Markdown
Member

@golanglemonade golanglemonade commented Mar 16, 2026

This supports new permissions updates: theopenlane/core#2398

adds parent_context for organization level tuples; related to theopenlane/core#2398
setup with a config that allows disbaling for backwards compatibility as well as configuring the skip and conditions via fga client config opts, e.g:

	suite.ofgaTF = fgatest.NewFGATestcontainer(context.Background(),
		fgatest.WithModuleFile(fgaModuleFile),
		fgatest.WithEnvVars(coreutils.GetDefaultFGAEnvs()),
		fgatest.WithVersion(version),
		fgatest.WithSkipParentContextKinds("organization", "user", "system"),
		fgatest.WithParentSkipConditions(fgax.ParentContextConditionConfig{Kind: "group", Name: "public_group", Context: map[string]any{"public": false}}),
	)

or via core config:

authz:
  parentcontextconditions:
    - kind: group
      name: public_group
      context:
        public: false
  parentcontextskipkinds: 
    - organization
    - user
    - system

this is a breaking change but I've allowed it to be configured with backwards compatibly; see the draft PR in core.

@github-actions github-actions Bot added the enhancement New feature or request label Mar 16, 2026
@sonarqubecloud
Copy link
Copy Markdown

@golanglemonade golanglemonade force-pushed the feat-parent-context-tuple branch from ced2a0d to b0c0f70 Compare May 5, 2026 04:11
@golanglemonade golanglemonade force-pushed the feat-parent-context-tuple branch from d02a398 to 1200f6c Compare May 14, 2026 06:21
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
@golanglemonade golanglemonade force-pushed the feat-parent-context-tuple branch from 1200f6c to a997d5f Compare May 14, 2026 06:30
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
@golanglemonade golanglemonade marked this pull request as ready for review May 14, 2026 07:17
@golanglemonade golanglemonade requested a review from a team as a code owner May 14, 2026 07:17
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@golanglemonade golanglemonade merged commit 519b585 into main May 18, 2026
15 checks passed
@golanglemonade golanglemonade deleted the feat-parent-context-tuple branch May 18, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants