Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.18%. Comparing base (01b45be) to head (918b1a5).
⚠️ Report is 9 commits behind head on master.

@@           Coverage Diff           @@
##           master    #3000   +/-   ##
=======================================
  Coverage   93.17%   93.18%           
=======================================
  Files          43       43           
  Lines        4501     4504    +3     
=======================================
+ Hits         4194     4197    +3     
  Misses        189      189           
  Partials      118      118           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

The description is little bit cryptic on first clance. It would be better if the situation/use-case would be explained in a "caveman manner"

I am using Decompress middleware along with the body limit middleware. This snippet

func main() {
	e := echo.New()

	e.Use(middleware.Decompress())
	e.Use(middleware.BodyLimit(4))

	e.POST("/", func(c *echo.Context) error {
		body, readErr := io.ReadAll(c.Request().Body)
		if readErr != nil {
			return readErr
		}
		return c.String(http.StatusOK, string(body))
	})

	if err := e.Start(":8080"); err != nil {
		slog.Error("Failed to start server", "error", err)
	}
}

does not work because the Decompress middleware leaves req.ContentLength unchanged after decompressing and this will trigger BodyLimit middleware to deny the request, although the compressed body is actually smaller than ContentLength reports.

At the moment and tested also with

printf "ok" | gzip | curl   -X POST   -H "Content-Type: text/plain"   -H "Content-Encoding: gzip"   --data-binary @-   http://localhost:8080/

Currently the result is incorrect:

{"message":"Request Entity Too Large"}

Expected would be:

ok

just nitpicking here as it is not easy sometimes to understand what the actual use-case is, and fixing (accepting PRs) something that you do not understand can break things on some other place.

Please not to take this as me being mean. I am trying keep things simple and avoid unessesary complexity if possible.

Thanks for the clarification, and sorry for the churn. I trimmed this back to the BodyLimit regression: Decompress swaps in the decoded body, but ContentLength still reflected the compressed request.

I removed the Content-Encoding, GetBody, and multiple-encoding changes to keep this focused.