Skip to content

Conversation

@therepanic
Copy link
Contributor

Closes: gh-17293

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 3, 2025
@jzheaux jzheaux added in: acl An issue in spring-security-acl status: blocked An issue that's blocked on an external project change type: breaks-passivity A change that breaks passivity with the previous release and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 23, 2025
@jzheaux jzheaux self-assigned this Jul 23, 2025
@jzheaux
Copy link
Contributor

jzheaux commented Jul 23, 2025

@therepanic, thank you for the PR!

I've been receiving feedback to keep the access API around for more time. Originally I thought we would still remove the ACL implementations, and upon further review, I'd prefer to remove the entire access API at the same time.

An intermediate step I'm researching is moving these and other access classes into a separate spring-security-access module.

Given these two possibilities, I've marked the PR as blocked for the moment.

@jzheaux
Copy link
Contributor

jzheaux commented Sep 10, 2025

@therepanic, it was decided to keep these for now, just in a different module, as explained above.

Thank you for contributing this! Let's revisit again in Spring Security 8. To keep the pull request list a little cleaner, I'm going to close this for the time being. We can reopen this once we are ready to remove the Access API altogether, or we can create a new PR.

I've created #17868 to track this.

@jzheaux jzheaux closed this Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: acl An issue in spring-security-acl status: blocked An issue that's blocked on an external project change type: breaks-passivity A change that breaks passivity with the previous release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove ACL access implementations in favor of AclPermissionEvaluator

3 participants