⬆ Bump Starlette to version 0.22.0 to fix bad encoding for query parameters in TestClient

[azogue]

Update starlette to v0.22.0 to fix issue with TestClient doing an extra unquote in the URL query

(encode/starlette#1952 and encode/starlette#1953), introduced with the (tests)-Breaking change in v0.21.0 with the swap from requests.Session to httpx.Client for the TestClient

closes #5646

[github-actions]

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

[iudeen]

Nice.

And thanks for the test too :)

[papb]

This will be great for the fix on GZipMiddleware too!

[lovasoa]

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

[azogue]

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 ==

[tiangolo]

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. 🚀

[papb]

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.