Skip to content

Replace java.security.AccessController with org.opensearch.secure_sm.AccessController#1974

Merged
ps48 merged 2 commits intoopensearch-project:mainfrom
cwperks:replace-access-controller
Feb 17, 2026
Merged

Replace java.security.AccessController with org.opensearch.secure_sm.AccessController#1974
ps48 merged 2 commits intoopensearch-project:mainfrom
cwperks:replace-access-controller

Conversation

@cwperks
Copy link
Copy Markdown
Member

@cwperks cwperks commented Feb 7, 2026

Description

Replace java.security.AccessController with org.opensearch.secure_sm.AccessController.

java.security.AccessController is marked for removal and needs to be replaced.

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…AccessController

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@ps48
Copy link
Copy Markdown
Member

ps48 commented Feb 9, 2026

@cwperks thanks for the PR. Is there a campaign to do this migration for all the opensearch backend plugins?

Can you please look at the build failure, should be a small linter fix:


> Task :ktlint
/Users/runner/work/observability/observability/src/main/kotlin/org/opensearch/observability/security/SecurityAccess.kt:18:88: Function body should be replaced with body expression (standard:function-expression-body)
21:10:53.539 [main] WARN com.pinterest.ktlint.cli.internal.KtlintCommandLine -- Lint has found errors than can be autocorrected using 'ktlint --format'

Summary error count (descending) by rule:
  standard:function-expression-body: 1

> Task :ktlint FAILED

> Task :compileKotlin
w: file:///Users/runner/work/observability/observability/src/main/kotlin/org/opensearch/observability/index/ObservabilityIndex.kt:101:22 'fun mapping(p0: String!, p1: XContentType!): CreateIndexRequest!' is deprecated. Deprecated in Java.
w: file:///Users/runner/work/observability/observability/src/main/kotlin/org/opensearch/observability/index/ObservabilityIndex.kt:102:22 'fun settings(p0: String!, p1: XContentType!): CreateIndexRequest!' is deprecated. Deprecated in Java.


FAILURE: Build failed with an exception.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@cwperks
Copy link
Copy Markdown
Member Author

cwperks commented Feb 9, 2026

Ran ./gradlew ktlintFormat and pushed.

@ps48 ps48 merged commit c1cff28 into opensearch-project:main Feb 17, 2026
8 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants