Skip to content

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented May 6, 2025

Now that SecurityManager is no longer used, doPrivileged is no longer necessary. This commit removes uses of it from core and server

Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
@rjernst rjernst requested a review from a team as a code owner May 6, 2025 20:08
@rjernst rjernst added auto-backport Automatically create backport pull requests when merged v8.19.0 v9.1.0 :Core/Infra/Entitlements Entitlements infrastructure labels May 6, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label May 6, 2025
Copy link
Contributor

@ldematte ldematte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rjernst rjernst merged commit 9537388 into elastic:main May 7, 2025
17 checks passed
@rjernst rjernst deleted the sm/remove_do_priv branch May 7, 2025 14:24
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.19 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 127781

ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request May 9, 2025
Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request May 9, 2025
Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request May 12, 2025
Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Jun 4, 2025
Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
elasticsearchmachine pushed a commit that referenced this pull request Jun 4, 2025
Now that SecurityManager is no longer used, doPrivileged is no longer
necessary. This commit removes uses of it from core and server
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Core/Infra/Entitlements Entitlements infrastructure >refactoring Team:Core/Infra Meta label for core/infra team v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants