Skip to content

Conversation

@rjernst
Copy link
Member

@rjernst rjernst commented May 6, 2025

The SecureMockMaker is a way for mockito to be allowed SM permissions for proxying. Since the Security Manager is no longer used, secure mocks are no longer needed. This commit removes them.

The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
@rjernst rjernst added >refactoring 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 elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label May 6, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@rjernst rjernst merged commit 1a1763c into elastic:main May 6, 2025
17 checks passed
@rjernst rjernst deleted the sm/remove_secure_mock branch May 6, 2025 19:31
@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 127773

parkertimmins pushed a commit to parkertimmins/elasticsearch that referenced this pull request May 7, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
ywangd pushed a commit to ywangd/elasticsearch that referenced this pull request May 9, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
afoucret pushed a commit to afoucret/elasticsearch that referenced this pull request May 9, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request May 12, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
rjernst added a commit to rjernst/elasticsearch that referenced this pull request Jun 4, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
elasticsearchmachine pushed a commit that referenced this pull request Jun 4, 2025
The SecureMockMaker is a way for mockito to be allowed SM permissions
for proxying. Since the Security Manager is no longer used, secure mocks
are no longer needed. This commit removes them.
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