Enable RASP by default

[estringana]

Description

Enable RASP by default

APPSEC-56457

Reviewer checklist

• Test coverage seems ok.
• Appropriate labels assigned.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.95%. Comparing base (4378c26) to head (9f50be2).

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #3082      +/-   ##
============================================
- Coverage     74.74%   72.95%   -1.80%     
  Complexity     2790     2790              
============================================
  Files           112      139      +27     
  Lines         11042    15280    +4238     
  Branches          0     1047    +1047     
============================================
+ Hits           8253    11147    +2894     
- Misses         2789     3580     +791     
- Partials          0      553     +553     

Flag	Coverage Δ
appsec-extension	68.28% <ø> (?)
tracer-php	74.74% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
appsec/src/extension/configuration.h	100.00% <ø> (ø)

... and 26 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4378c26...9f50be2. Read the comment docs.

[pr-commenter]

Benchmarks [ appsec ]

Benchmark execution time: 2025-02-12 10:03:12

Comparing candidate commit 9f50be2 in PR branch estringana/enable-rasp-by-default with baseline commit 4378c26 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics.

[Anilm3]

Found 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics.

This is unexpected...

[estringana]

Found 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics.

This is unexpected...

It's not that unexpected. All tests which were consuming time for rasp were already enabled. There may not be many tests which are actually using the wrapped function and that's the reason why there was no impact