Package "stream" is misleadingly named about oxy HOT 9 CLOSED

great catch, I thought about, buffer is too generic though, I promise to rename it if you come up with a great name :-)

from oxy.

StreamBuffer

from oxy.

@bradgignac too long

from oxy.

streambuf?

from oxy.

oxybuf?

from oxy.

retry? gather?

from oxy.

buffer/buffer.go - Buffer request into memory or disk

Reads the entire request and response into buffer, optionally buffering it to disk for large requests.
Checks the buffer.Mem and buffer.Max for the requests and responses, rejecting in case if the buffer size was exceeded.
Changes request content-transfer-encoding from chunked and provides total buffer size to the handlers.

Ex.

buffer up to 2MB in memory

buffer.MemRequestBodyBytes(2 * 1024 * 1024)

buffer sounds great! when I'm reading the code as buffer.

code shift for readability

• stream limits to buffer size

haven't read all the code yet, but I will PR the shift in code if sounds good in use

from oxy.

@archisgore renamed the package, maybe this could be closed?

from oxy.

I'm closing it. Please reopen if something is amiss.

from oxy.