Expand appsec integrations to Laminas Framework

[estringana]

Description

Reviewer checklist

• Test coverage seems ok.
• Appropriate labels assigned.

[datadog-prod-us1-4]

⚠️ Tests

✨ Fix all issues with BitsAI or with Cursor

⚠️ Warnings

❄️ 4 New flaky tests detected

testLoggedInCalls from tests/Integrations/Laminas/Mvc/V3_3.DDTrace\Tests\Integrations\Laminas\Mvc\V3_3\AutomatedLoginEventsTest     (Fix with Cursor)

DDTrace
Failed asserting that 0 matches expected 1.

tests/Integrations/Laminas/Mvc/AutomatedLoginEventsTestSuite.php:122
tests/Common/RetryTraitVersionGeneric.php:28
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

testUserLoginFailureEvent from tests/Integrations/Laminas/Mvc/V3_3.DDTrace\Tests\Integrations\Laminas\Mvc\V3_3\AutomatedLoginEventsTest     (Fix with Cursor)

DDTrace
Failed asserting that 0 matches expected 1.

tests/Integrations/Laminas/Mvc/AutomatedLoginEventsTestSuite.php:72
tests/Common/RetryTraitVersionGeneric.php:28
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

testUserLoginSuccessEvent from tests/Integrations/Laminas/Mvc/V3_3.DDTrace\Tests\Integrations\Laminas\Mvc\V3_3\AutomatedLoginEventsTest     (Fix with Cursor)

DDTrace
Failed asserting that 0 matches expected 1.

tests/Integrations/Laminas/Mvc/AutomatedLoginEventsTestSuite.php:58
tests/Common/RetryTraitVersionGeneric.php:28
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

testUserSignUp from tests/Integrations/Laminas/Mvc/V3_3.DDTrace\Tests\Integrations\Laminas\Mvc\V3_3\AutomatedLoginEventsTest     (Fix with Cursor)

DDTrace
Failed asserting that 0 matches expected 1.

tests/Integrations/Laminas/Mvc/AutomatedLoginEventsTestSuite.php:88
tests/Common/RetryTraitVersionGeneric.php:28
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

View all

🧪 13 Tests failed

testSearchPhpBinaries from integration.DDTrace\Tests\Integration\PHPInstallerTest     (Fix with Cursor)

Risky Test
phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:60

testSimplePushAndProcess from laravel-58-test.DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest     (Fix with Cursor)

DDTrace\Tests\Integrations\Laravel\V5_8\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 69c25e6400000000608d47a87ad9f71a
tid: 69c25e6400000000
hexProcessTraceId: 608d47a87ad9f71a
hexProcessSpanId: 949f1568c7f88a0d
processTraceId: 6957295788298467098
processSpanId: 10709301978686065165

phpvfscomposer://tests/vendor/phpunit/phpunit/phpunit:106

testSimplePushAndProcess from laravel-8x-test.DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest     (Fix with Cursor)

DDTrace\Tests\Integrations\Laravel\V8_x\QueueTest::testSimplePushAndProcess
Test code or tested code printed unexpected output: spanLinksTraceId: 69c25df1000000007c1dda7c16ed61a3
tid: 69c25df100000000
hexProcessTraceId: 7c1dda7c16ed61a3
hexProcessSpanId: fca6e6f4baa8ddb4
processTraceId: 8943544661523128739
processSpanId: 18205492482469846452

View all

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 7a37a42 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!}

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.78%. Comparing base (78ecdb0) to head (7a37a42).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3716      +/-   ##
==========================================
- Coverage   68.81%   68.78%   -0.04%     
==========================================
  Files         166      166              
  Lines       19030    19030              
  Branches     1797     1797              
==========================================
- Hits        13096    13090       -6     
- Misses       5121     5127       +6     
  Partials      813      813              

Flag	Coverage Δ
helper-rust-integration	78.79% <ø> (-0.06%)	⬇️
helper-rust-unit	49.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 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 78ecdb0...7a37a42. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pr-commenter]

Benchmarks [ tracer ]

Benchmark execution time: 2026-03-23 16:32:35

Comparing candidate commit f9ebd59 in PR branch estringana/add-laminas-appsec-integration with baseline commit 78ecdb0 in branch master.

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

scenario:MessagePackSerializationBench/benchMessagePackSerialization

• 🟩 execution_time [-6.859µs; -5.581µs] or [-6.113%; -4.973%]

scenario:MessagePackSerializationBench/benchMessagePackSerialization-opcache

• 🟩 execution_time [-3.543µs; -2.397µs] or [-3.213%; -2.174%]

scenario:PHPRedisBench/benchRedisOverhead

• 🟥 execution_time [+30.284µs; +59.801µs] or [+3.852%; +7.606%]

scenario:SamplingRuleMatchingBench/benchRegexMatching1-opcache

• 🟩 execution_time [-9.770µs; -9.517µs] or [-87.318%; -85.061%]

[github-actions]

Snapshots difference summary

The following differences have been observed in committed snapshots. It is meant to help the reviewer.
The diff is simplistic, so please check some files anyway while we improve it.

If you need to update snapshots, please refer to CONTRIBUTING.md