Codecov Report

Merging#1138 intomaster willnot change coverage by%.
The diff coverage isn/a.

@@            Coverage Diff            @@##            master     #1138   +/-   ##=========================================  Coverage   100.00%   100.00%           =========================================  Files          223       223             Lines         6642      6642           =========================================  Hits          6642      6642

Continue to review full report at Codecov.

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

is it possible still wrong ?
should be starlette.middleware.gzip rather than fastapi.middleware.gzip

@bilencekic based on the content inhttps://github.com/tiangolo/fastapi/blob/master/fastapi/middleware/gzip.py which imports fromstarlette.middleware.gzip it should work right?

@tiangolo Could you have a look at this. I've seen 4 other PR fixing the same issue ^^

@arimbr yes correct it is working i am using like that but documentation has changed to
from fastapi.middleware.trustedhost import TrustedHostMiddleware
i think GZIP is also embedded to this middleware now.

Right! Thanks, It is just syntactic sugar introduced in#1064 where this issue was also introduced.

Great, thank you@arimbr ! 🔍 🕵️‍♂️ 🍰 🚀

And thanks for the discussion here everyone 👏 🍰 ☕