scout_apm_python icon indicating copy to clipboard operation
scout_apm_python copied to clipboard

Check `ignore` value for legacy configuration

Open quinnmil opened this issue 8 months ago • 1 comments

Changes

  • Fixes a bug where if the only sampling configuration set was the ignore value, we weren't checking for this in the any_sampling check.
  • Adds a test for this case.

quinnmil avatar May 06 '25 19:05 quinnmil

Realized that the changes here are redundant. This has already been supported via this:

        self.ignore_endpoints = set(
            config.value("ignore_endpoints") + config.value("ignore")
        )

So we're removing extra logic, but leaving in the new test.

quinnmil avatar May 06 '25 20:05 quinnmil