Skip to content

[9.1] Remove logging of trusted system classes to not risk causing ClassCircularityError. (#134431)#134445

Merged
elasticsearchmachine merged 1 commit intoelastic:9.1from
mosche:backport/9.1/pr-134431
Sep 10, 2025
Merged

[9.1] Remove logging of trusted system classes to not risk causing ClassCircularityError. (#134431)#134445
elasticsearchmachine merged 1 commit intoelastic:9.1from
mosche:backport/9.1/pr-134431

Conversation

@mosche
Copy link
Copy Markdown
Contributor

@mosche mosche commented Sep 10, 2025

Backports the following commits to 9.1:

…cularityError. (elastic#134431)

We've seen this previously (and predictably) in elastic#131539 due to logging Exceptions
in isTriviallyAllowed. elastic#133269 shows exactly the same symptoms.

Fixes elastic#133269
Fixes elastic#133267
Fixes elastic#133268
@mosche mosche added the :Core/Infra/Entitlements Entitlements infrastructure label Sep 10, 2025
@mosche mosche requested a review from a team as a code owner September 10, 2025 11:47
@mosche mosche added >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Core/Infra Meta label for core/infra team labels Sep 10, 2025
@elasticsearchmachine elasticsearchmachine merged commit d4f804e into elastic:9.1 Sep 10, 2025
34 checks passed
@mosche mosche deleted the backport/9.1/pr-134431 branch September 10, 2025 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Entitlements Entitlements infrastructure >non-issue Team:Core/Infra Meta label for core/infra team v9.1.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants