open-telemetry / opentelemetry-java

OpenTelemetry Java SDK
https://opentelemetry.io
Apache License 2.0
1.97k stars 814 forks source link

Fix ConcurrentModificationException #6732 #6746

Open neugartf opened 2 days ago

neugartf commented 2 days ago

Could be nicer with jdk > 8, but that's how it is :). I added a filter to make sure only string key/values from Properties are read into our logic.

linux-foundation-easycla[bot] commented 2 days ago

CLA Signed

The committers listed above are authorized under a signed CLA.

codecov[bot] commented 2 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.07%. Comparing base (697b4e0) to head (329c05d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #6746 +/- ## ============================================ - Coverage 90.08% 90.07% -0.01% + Complexity 6464 6463 -1 ============================================ Files 718 718 Lines 19528 19529 +1 Branches 1923 1923 ============================================ Hits 17591 17591 - Misses 1348 1350 +2 + Partials 589 588 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.