Skip to content

Feat/security : Secure Task Management with Custom Voters & Access Control#42

Merged
Mike031289 merged 4 commits into
developfrom
feat/security-admin-access
Jul 16, 2026
Merged

Feat/security : Secure Task Management with Custom Voters & Access Control#42
Mike031289 merged 4 commits into
developfrom
feat/security-admin-access

Conversation

@Mike031289

Copy link
Copy Markdown
Owner

Resolve Issue: Complete Task Security & Authorization Flow

This PR implements fine-grained authorization and secures the entire task management lifecycle—from database fixtures and configuration to backend controllers and frontend views.

Key Changes

  • Security & Configuration:
    • Created custom Symfony Voters for precise permission checks.
    • Secured application routes via access_control in security.yml.
    • Configured dependency injection in services.yml.
  • Backend (TaskController.php): Applied security checks to restrict task creation, modification, and deletion.
  • Frontend (list.html.twig): Integrated conditional rendering (UI checks) to display actions based on user permissions.
  • Database & QA:
    • Updated Fixtures to set up proper test users and security roles.
    • Updated TaskControllerTest.php to validate all secure/unsecure access scenarios.

Verification

  • Ran and validated the updated test suite:
    php bin/phpunit tests/AppBundle/Controller/TaskControllerTest.php

…ccess controls

- Created custom security voters in AppBundle/Security to handle fine-grained permissions
- Configured service definitions in services.yml for the new security components
- Secured application routes via access_control rules in security.yml
…onal tests

- Applied security and authorization checks in TaskController
- Updated TaskControllerTest to cover new access control scenarios
@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 2 medium

Alerts:
⚠ 2 issues (≤ 0 issues of at least minor severity)

Results:
2 new issues

Category Results
UnusedCode 2 medium

View in Codacy

🟢 Metrics 8 complexity · 0 duplication

Metric Results
Complexity 8
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@Mike031289
Mike031289 merged commit a581b9d into develop Jul 16, 2026
1 of 2 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in @Mike031289's ToDo-Co Jul 16, 2026
@Mike031289
Mike031289 deleted the feat/security-admin-access branch July 16, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Extra attention is needed security

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant