Skip to content

feat: add per-site /{site}/CONFIG permission for site config#293

Open
bosschaert wants to merge 2 commits into
mainfrom
site-config
Open

feat: add per-site /{site}/CONFIG permission for site config#293
bosschaert wants to merge 2 commits into
mainfrom
site-config

Conversation

@bosschaert

Copy link
Copy Markdown
Contributor

Description

Introduce a per-site config permission keyword /{site}/CONFIG that governs reading/writing site config (/config/{org}/{site}/...), mirroring how the org-level CONFIG keyword governs org config (/config/{org}).

Related Issue

Fixes: #292

How Has This Been Tested?

New unit tests

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Introduce a per-site config permission keyword `/{site}/CONFIG` that
governs reading/writing site config (/config/{org}/{site}/...), mirroring
how the org-level `CONFIG` keyword governs org config (/config/{org}).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Site config access is governed by the per-site /{site}/CONFIG keyword only
when such a rule is present in the permissions sheet. When no /{site}/CONFIG
rule is specified, site config access falls back to the org-level CONFIG rule.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@bosschaert bosschaert requested review from hannessolo and mhaack June 16, 2026 15:28
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.

Create a new permission to govern site-specific config access

3 participants