fix(profiling): mismatch in utf8 handling

[morrisonlevi]

Description

Happens in timeline filename for zend_error_observer.

I found this mismatch while doing something else. I asked Florian about it and he prepared a different way to fix it in #2988. I thought I'd push mine up too so we can compare.

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 74.82%. Comparing base (8273bb7) to head (f013c40).
Report is 1 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2990      +/-   ##
============================================
+ Coverage     72.90%   74.82%   +1.91%     
  Complexity     2741     2741              
============================================
  Files           137      110      -27     
  Lines         14978    10863    -4115     
  Branches       1016        0    -1016     
============================================
- Hits          10920     8128    -2792     
+ Misses         3506     2735     -771     
+ Partials        552        0     -552     

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

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

see 27 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 8273bb7...f013c40. Read the comment docs.

[pr-commenter]

Benchmarks [ profiler ]

Benchmark execution time: 2024-12-12 07:08:02

Comparing candidate commit f013c40 in PR branch levi/filename-fix with baseline commit 8273bb7 in branch master.

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

[realFlowControl]

LGTM 🎉