Fixed a problem that returned wrong content-encoding when the gzip co…

[orisano]

I fixed a problem that returned wrong content-encoding when the gzip compressed content was empty.

gzip.Writer writes a gzip header if Write is called even once, and does not return the length of the header.
we need to record the Write call and make sure that the gzip header is written.

[codecov]

Codecov Report

Merging #1921 (73a266d) into master (58366f9) will increase coverage by 0.35%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1921      +/-   ##
==========================================
+ Coverage   90.97%   91.33%   +0.35%     
==========================================
  Files          32       33       +1     
  Lines        2792     2872      +80     
==========================================
+ Hits         2540     2623      +83     
+ Misses        161      159       -2     
+ Partials       91       90       -1     

Impacted Files	Coverage Δ
middleware/compress.go	84.00% <100.00%> (-0.32%)	⬇️
middleware/jwt.go	90.26% <0.00%> (ø)
middleware/timeout.go	100.00% <0.00%> (ø)
middleware/rate_limiter.go	100.00% <0.00%> (ø)
middleware/request_logger.go	97.59% <0.00%> (ø)
router.go	95.72% <0.00%> (+0.02%)	⬆️
echo.go	94.15% <0.00%> (+0.32%)	⬆️
middleware/key_auth.go	96.92% <0.00%> (+0.37%)	⬆️
middleware/request_id.go	85.71% <0.00%> (+1.50%)	⬆️
middleware/decompress.go	96.29% <0.00%> (+7.92%)	⬆️

---

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 58366f9...73a266d. Read the comment docs.

[orisano]

what is the status of this pr? @aldas

[orisano]

ping @aldas

[aldas]

LGTM, sorry for delay, these Github messages get sometimes buried in my mailbox