fix(cache): don't use _originalPath

[danielroe]

🔗 Linked issue

Dropped in unjs/h3#506

❓ Type of change

• 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

I think caching on path probably makes sense but we could also use event.node.req.originalUrl || event.node.req.url || to match current behaviour.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[codecov]

Codecov Report

Merging #1576 (c8250c0) into main (12d0e47) will decrease coverage by 0.06%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1576      +/-   ##
==========================================
- Coverage   76.00%   75.95%   -0.06%     
==========================================
  Files          74       74              
  Lines        7710     7710              
  Branches      760      760              
==========================================
- Hits         5860     5856       -4     
- Misses       1848     1851       +3     
- Partials        2        3       +1     

see 1 file with indirect coverage changes

[pi0]

Thanks ltgm. two notes:

• We might later simplify caching strategy to use event.path only. Need to double check all places
• Will also have to check why tests didn't caught it.

[danielroe]

I think tests did catch it: https://github.com/unjs/nitro/actions/runs/5860687198/job/15889277077