why is echo.Context used in middleware/body_limit.go?

[imacks]

Struct limitedReader has field context of type echo.Context, which gets associated with a request via its Reset method. Why do we need the context field at all? It doesn't look like limitedReader actually uses the context at all. Is there some GC related issue?

[aldas]

no, for these cases you should check the history of that code. It seems that back in the day that context was used but that usage has been removed and that field was forgotten there.

[imacks]

Tks for confirming! I wasn't sure if it's something I overlooked. If you like I am happy to do a PR to remove the extra field :)

[aldas]

if you want to and have time, send us the PR.

[eroncastro]

Since #2483 was merged, maybe this issue could be closed.