ref(wsgi): Update _werkzeug vendor to newer version

[mgaligniana]

Fixes GH-3516

[codecov]

Codecov Report

❌ Patch coverage is 44.44444% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.41%. Comparing base (5a122b5) to head (ba2c350).

Files with missing lines	Patch %	Lines
sentry_sdk/_werkzeug.py	44.44%	11 Missing and 4 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4793      +/-   ##
==========================================
- Coverage   84.47%   83.41%   -1.07%     
==========================================
  Files         158      158              
  Lines       16506    16512       +6     
  Branches     2865     2864       -1     
==========================================
- Hits        13944    13773     -171     
- Misses       1712     1889     +177     
  Partials      850      850              

Files with missing lines	Coverage Δ
sentry_sdk/_werkzeug.py	54.54% <44.44%> (+6.39%)	⬆️

... and 8 files with indirect coverage changes

[sentrivana]

Thanks for the PR @mgaligniana -- I think the HTTP_X_FORWARDED_HOST handling needs to be added manually on top of the vendored in code. As far as I can tell we also did that in the current version as the original upstream 1.0.1 version also doesn't contain it.

[mgaligniana]

Ok, Seer 🎉 liked!

But codecov not, should I add new tests for the new vendor?

Edited: I think yes because Anton said:

Also make sure this code is tested to have a high test coverage. (maybe by vendoring in the related tests from Werkzeug?)

But do you think I should copy-paste the test from werkzeug to get_host for example?

[sentrivana]

But do you think I should copy-paste the test from werkzeug to get_host for example?

Tests would be great. I think the one you linked would be good, plus something custom for the HTTP_X_FORWARDED_HOST stuff.

[mgaligniana]

So adding that now we have

[mgaligniana]

There we go!

[mgaligniana]

Hi @sentrivana, just following up on this! In case you need any changes from my side! Thank you!

[sentrivana]

Hey @mgaligniana, sorry for dropping the ball on this, I'll get to reviewing eventually -- no need for you to wait though, I can just apply any changes myself. Thanks for your work!

[mgaligniana]

Perfect, no problem!! Thank you!!

[github-actions]

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

---

New Features ✨

• feat(span-streaming): Add experimental trace_lifecycle switch (1) by sentrivana in #5397

Bug Fixes 🐛

• fix(arq): handle settings_cls passed as keyword argument by nc9 in #5393
• fix(dramatiq): cleanup isolated scope and transaction when message is skipped by frankie567 in #5346
• fix(google-genai): deactivate google genai when langchain is used by shellmayr in #5389

Internal Changes 🔧

• ref(wsgi): Update _werkzeug vendor to newer version by mgaligniana in #4793

• ci: migration to the new codecov action by MathurAditya724 in #5392

---

_{🤖 This preview updates automatically when you update the PR.}

[sentrivana]

Hey, I'm sorry but I'll close this PR after all. It's drifted from master a lot (which is my fault), and I'm also a bit concerned about refactoring this code just for the sake of refactoring it (this is also our fault for making the issue in the first place), since it seems to have been working fine until now.

[mgaligniana]

Don't worry @sentrivana and thanks for the explanation! I hope I can get another ticket! 🚀