📝 Docs preview for commit d42a417 at: https://6377916021146403d9d10c6f--fastapi.netlify.app

This will be great for the fix on GZipMiddleware too!

Thank you @azogue ! I spent forever tracking back a broken test to this issue ! Is there something missing for this PR to be merged ?

Is there something missing for this PR to be merged ?

@lovasoa, I don't know :(

If there is, I could do it if pointed to 🥺

Release notes are filled with GH Actions, I think, and there is no 'milestone' tracking to add it for the next patch

I suppose it will be merged soon 🤞

We are using fastAPI in production with 0.85.1, waiting for the fix to update it, after adapting the test suite with the TestClient change.

The only thing I would do different is to unpin the starlette dependency, at least to ^0.22.0, instead of setting a fixed version each time, but I can understand the reasons behind, so I just changed the version, maintaining the hard link ==

Great job @azogue! Thanks for adding a test for it! 🙇

And thanks everyone for the interactions, especially @iudeen that I see you reviewed it, and added a comment letting me know you checked the code and test 😉 🍪

This is available in FastAPI 0.88.0, just released. 🚀

at least to ^0.22.0,

@azogue I would be in favor of that too, but you know that means >=0.22.0 && <0.23.0, right? If it was previously set to ^0.21.0 instead of ==0.21.0, it would still need the update anyway, because 0.22.0 is not covered by ^0.21.0.