Skip to content

fix: Replace deprecated ACL.SYSTEM with ACL.SYSTEM2 in doCheckCredentialsId() and migrateCredentials()#1505

Open
ArpanC6 wants to merge 1 commit intojenkinsci:mainfrom
ArpanC6:fix/acl-system-v3
Open

fix: Replace deprecated ACL.SYSTEM with ACL.SYSTEM2 in doCheckCredentialsId() and migrateCredentials()#1505
ArpanC6 wants to merge 1 commit intojenkinsci:mainfrom
ArpanC6:fix/acl-system-v3

Conversation

@ArpanC6
Copy link

@ArpanC6 ArpanC6 commented Mar 17, 2026

Summary

Replace deprecated ACL.SYSTEM with ACL.SYSTEM2 in two methods in MailAccount.java.

Changes

  • Line 173: Replaced ACL.SYSTEM with ACL.SYSTEM2 in doCheckCredentialsId()
  • Line 313: Replaced ACL.SYSTEM with ACL.SYSTEM2 in migrateCredentials()

Testing

No functional change, only deprecation fix.
This follows up on the same fix applied in #1494 for doFillCredentialsIdItems().

@ArpanC6 ArpanC6 requested a review from a team as a code owner March 17, 2026 09:39
@ArpanC6
Copy link
Author

ArpanC6 commented Mar 17, 2026

The CI failures are pre-existing issues unrelated to this change:

  1. Jenkins Security Scan: Build fails in MailAccount.java due to org.acegisecurity.Authentication → org.springframework.security.core.Authentication incompatibility (pre-existing, not introduced here)
  2. JUnit step failure: Pre-existing issue on linux-25, also present on other PRs

This PR only replaces ACL.SYSTEM with ACL.SYSTEM2 in two methods — no functional change.

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.

2 participants