fix: exception recording for Symfony sub-requests in instrumentation

[Apoorva64]

Fixes open-telemetry/opentelemetry-php#1844

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: Apoorva64 / name: Apoorva Srinivas Appadoo (8df53c1)

[Apoorva64]

@bobstrecansky could you take a look a this issue/PR?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.44%. Comparing base (a3c02c8) to head (8df53c1).

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #494      +/-   ##
============================================
- Coverage     83.46%   83.44%   -0.02%     
  Complexity     2330     2330              
============================================
  Files           149      149              
  Lines          8791     8791              
============================================
- Hits           7337     7336       -1     
- Misses         1454     1455       +1     

Flag	Coverage Δ
Aws	93.37% <ø> (ø)
Context/Swoole	0.00% <ø> (ø)
Exporter/Instana	49.80% <ø> (ø)
Instrumentation/AwsSdk	82.00% <ø> (ø)
Instrumentation/CakePHP	20.42% <ø> (ø)
Instrumentation/CodeIgniter	79.31% <ø> (ø)
Instrumentation/Curl	86.88% <ø> (-0.28%)	⬇️
Instrumentation/Doctrine	92.82% <ø> (ø)
Instrumentation/ExtAmqp	88.80% <ø> (ø)
Instrumentation/ExtRdKafka	86.13% <ø> (ø)
Instrumentation/Guzzle	76.25% <ø> (ø)
Instrumentation/HttpAsyncClient	78.94% <ø> (ø)
Instrumentation/IO	0.00% <ø> (ø)
Instrumentation/Laravel	71.65% <ø> (ø)
Instrumentation/MongoDB	76.84% <ø> (ø)
Instrumentation/MySqli	93.39% <ø> (ø)
Instrumentation/OpenAIPHP	86.71% <ø> (ø)
Instrumentation/PDO	85.67% <ø> (ø)
Instrumentation/PostgreSql	91.36% <ø> (ø)
Instrumentation/Psr14	77.41% <ø> (ø)
Instrumentation/Psr15	89.74% <ø> (ø)
Instrumentation/Psr16	97.43% <ø> (ø)
Instrumentation/Psr18	79.41% <ø> (ø)
Instrumentation/Psr3	67.70% <ø> (ø)
Instrumentation/Psr6	97.56% <ø> (ø)
Instrumentation/ReactPHP	99.41% <ø> (ø)
Instrumentation/Session	94.28% <ø> (ø)
Instrumentation/Slim	84.21% <ø> (ø)
Instrumentation/Yii	83.33% <ø> (ø)
Logs/Monolog	100.00% <ø> (ø)
Propagation/CloudTrace	90.69% <ø> (ø)
Propagation/Instana	98.07% <ø> (ø)
Propagation/ServerTiming	94.73% <ø> (ø)
Propagation/TraceResponse	94.73% <ø> (ø)
ResourceDetectors/Azure	91.66% <ø> (ø)
ResourceDetectors/Container	93.02% <ø> (ø)
ResourceDetectors/DigitalOcean	100.00% <ø> (ø)
Sampler/Xray	78.38% <ø> (ø)
Shims/OpenTracing	92.99% <ø> (ø)
SqlCommenter	95.58% <ø> (ø)
Symfony	88.14% <ø> (ø)
Utils/Test	87.79% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file 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 a3c02c8...8df53c1. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[brettmc]

LGTM, @Apoorva64 - can you write or update a test to show it working?

[jvdlaar]

I was looking into adding these tests such that we can get this merged. But at the moment I can't even get the dev requirements installed. Seems like phan is causing an impossible combination of dependencies. Any ideas on how to progress from here?