Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhance API Security #1430

Open
wants to merge 8 commits into
base: staging
Choose a base branch
from
Open

Enhance API Security #1430

wants to merge 8 commits into from

Conversation

mshakirdfe
Copy link
Contributor

What is the change?
Token based Authentication and Role Based Authorisation
Why do we need the change?
Improve security posture
What is the impact?

Azure DevOps Ticket
https://dev.azure.com/dfe-gov-uk/Academies-and-Free-Schools-SIP/_workitems/edit/186437
https://dev.azure.com/dfe-gov-uk/Academies-and-Free-Schools-SIP/_workitems/edit/188885

@DrizzlyOwl DrizzlyOwl removed their request for review December 20, 2024 09:50
@DrizzlyOwl
Copy link
Collaborator

There are too many files for me to review and I am not an expert in .NET so I am removing myself as a reviewer

@FrostyApeOne FrostyApeOne reopened this Dec 20, 2024
@mshakirdfe mshakirdfe force-pushed the feature/188885-API-security-enhancement branch from 05cd01a to a89d6cb Compare January 2, 2025 09:28
@mshakirdfe mshakirdfe force-pushed the feature/188885-API-security-enhancement branch from f8f84ea to 743668e Compare January 6, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FrostyApeOne FrostyApeOne FrostyApeOne left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mshakirdfe @DrizzlyOwl @FrostyApeOne