Skip to content

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

Merged
elasticsearchmachine merged 2 commits intoelastic:8.19from
mosche:backport/8.19/pr-134431
Sep 19, 2025
Merged

[8.19] Remove logging of trusted system classes to not risk causing ClassCircularityError. (#134431)#134444
elasticsearchmachine merged 2 commits intoelastic:8.19from
mosche:backport/8.19/pr-134431

Conversation

@mosche
Copy link
Copy Markdown
Contributor

@mosche mosche commented Sep 10, 2025

Backports the following commits to 8.19:

…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 :Core/Infra/Entitlements Entitlements infrastructure >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 ec659a9 into elastic:8.19 Sep 19, 2025
23 checks passed
@mosche mosche deleted the backport/8.19/pr-134431 branch September 19, 2025 13:59
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 v8.19.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants