fix(rc): rework remote configuration handling by dmehala · Pull Request #234 · DataDog/dd-trace-cpp

zacharycmontoya

zacharycmontoya

zacharycmontoya

Base automatically changed from dmehala/fix-rc-sampling-2 to main

September 3, 2025 09:38
This commit refactors the way remote configurations are applied
to the tracer. Previously, if a remote configuration contained both
valid and invalid fields, the valid parts could be partially applied,
potentially leading to inconsistent behavior.

Now, the configuration is only applied if it is fully valid. Any invalid
configuration is entirely rejected. This constitutes a breaking change.

This also report errors when an invalid configuration is rejected.

Changes:
  - Add fuzz test and input corpus for Remote Configuration.
  - Fix a crash when telemetry methods are invoked before intializing
    the module.
  - Improve unit test coverage of remote configuration.
Co-authored-by: Zach Montoya <zach.montoya@datadoghq.com>

@dmehala dmehala deleted the dmehala/fix-rc-report-conf-status branch

September 4, 2025 09:35

ygree