Skip to content

feat: Authorization RBAC Enhancement#987

Open
rubenvdlinde wants to merge 3 commits intodevelopmentfrom
fix/authorization-rbac-enhancement
Open

feat: Authorization RBAC Enhancement#987
rubenvdlinde wants to merge 3 commits intodevelopmentfrom
fix/authorization-rbac-enhancement

Conversation

@rubenvdlinde
Copy link
Contributor

@rubenvdlinde rubenvdlinde commented Mar 24, 2026

Summary

Context

This PR was previously merged and reverted along with other PRs while investigating a development branch issue.

Changes

  • Restores AuthorizationAuditService, PermissionHandler RBAC logic, MagicRbacHandler updates
  • Restores RegistersController and SchemasController RBAC integration
  • Restores OasService RBAC scope generation
  • Restores PermissionMatrix.vue settings view
  • Restores all OpenSpec specs and change archive for the RBAC enhancement
  • Restores unit tests: AuthorizationAuditServiceTest, PermissionHandlerRbacTest
  • Fixes ESLint error: v-for + v-if in PermissionMatrix.vue replaced with computed property registersWithBulkActions

This reverts commit 2d232bf.

# Conflicts:
#	openspec/changes/archive/2026-03-21-auth-system/specs/auth-system/spec.md
#	openspec/changes/archive/2026-03-21-rbac-zaaktype/specs/rbac-zaaktype/spec.md
#	openspec/changes/archive/2026-03-21-row-field-level-security/specs/row-field-level-security/spec.md
#	openspec/specs/rbac-scopes/spec.md
@github-actions
Copy link
Contributor

Quality Report

Repository ConductionNL/openregister
Commit 13abb0d
Branch 987/merge
Event pull_request
Generated 2026-03-24 11:23 UTC
Workflow Run https://github.com/ConductionNL/openregister/actions/runs/23486833169

Summary

Group Result
PHP Quality PASS
Vue Quality PASS
Security PASS
License PASS
PHPUnit SKIP
Newman SKIP
Playwright SKIP

PHP Quality

Tool Result
lint PASS
phpcs PASS
phpmd PASS
psalm PASS
phpstan PASS
phpmetrics PASS

Vue Quality

Tool Result
eslint PASS
stylelint PASS

Security

Ecosystem Result
composer PASS
npm PASS

License Compliance

Ecosystem Result
composer PASS
npm PASS

composer dependencies (147 total)

Metric Count
Approved (allowlist) 146
Approved (override) 1
Denied 0

npm dependencies (586 total)

Metric Count
Approved (allowlist) 585
Approved (override) 1
Denied 0

PHPUnit Tests

PHPUnit tests were not enabled for this run.

Integration Tests (Newman)

Newman integration tests were not enabled for this run.

E2E Tests (Playwright)

Playwright E2E tests were not enabled for this run.


Generated automatically by the Quality workflow.

Download the full PDF report from the workflow artifacts.

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.

1 participant