Codecov Report

Merging #1138 into master will not change coverage by %.
The diff coverage is n/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 by Codecov. Last update 670b643...f274f71. Read the comment docs.

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

@bilencekic based on the content in https://github.com/tiangolo/fastapi/blob/master/fastapi/middleware/gzip.py which imports from starlette.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 👏 🍰 ☕