Codecov Report

Merging#1462 intomaster willnot change coverage.
The diff coverage is100.00%.

@@            Coverage Diff            @@##            master     #1462   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          227       227             Lines         6807      6808    +1     =========================================+ Hits          6807      6808    +1

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update98bb9f1...694cb3e. Read thecomment docs.

You may also want to change the corresponding documentation as well.

You may also want to change the corresponding documentation as well.

I would but I don't know what corresponds.

https://github.com/tiangolo/fastapi/search?q=OAuth2PasswordRequestFormStrict

I always found it weird that the strict version was not exported, I think this is a good improvement. 👍

Thanks for your contribution@RichardHoekstra ! 🚀

So, it was actually intentional, I would expect (and suggest) everyone to use the non-strict version, as the strict version enforces clients to send a constant value doesn't add much, as it's just a constant.

Because of that, I didn't expose it directly.

But... I think this is like the 3th PR suggesting that change 😂

So I guess it's gonna be more intuitive if it's just there even though I don't expect many people to use it.

Thanks for the review@phy25 ! 🙇