search

gr3p1p3 / transparent-proxy

41 stars 17 forks source link

Wrong _response.complete definition: part 2 #42

Open fopinappb opened 1 year ago

fopinappb commented 1 year ago

complete response definition was only done for Transfer-Encoding (TE) and I opened #40 about it.

Fix ended like this:

https://github.com/gr3p1p3/transparent-proxy/blob/b09dc4c16f9e4a64db6377247bf716eef57be4b5/core/Session.js#L212-L220

Covering both TE and CL cases.

In the new HttpMirror class though, now only CL is handled https://github.com/gr3p1p3/transparent-proxy/blob/master/core/HttpMirror.js#L119-L124

gr3p1p3 commented 1 year ago

This should not be needed, because it happens on 'close' event!