Do not use ByteArray*Stream's for backing input/output streams

[jamezp]

We should not use ByteArrayInputStream and ByteArrayOutputStream for communication. This could cause issues for large payloads possibly crashing the JVM with an OOME.

[ronsigal]

I didn't know that. Is there a standard alternative?

…

On Thu, May 4, 2023, 1:04 PM James R. Perkins ***@***.***> wrote: We should not use ByteArrayInputStream and ByteArrayOutputStream for communication. This could cause issues for large payloads possibly crashing the JVM with an OOME. — Reply to this email directly, view it on GitHub <#2>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEDP4ZWLNLRX6YDRKJMSG3XEPOQ7ANCNFSM6AAAAAAXWBT664> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[jamezp]

Not currently. For the OutputStream we could use the org.jboss.resteasy.spi.EntityOutputStream. However, we don't currently have a matching InputStream variation. I could create something like this though.

[jamezp]

I spoke too soon :) If it works, I'd have to look, there is a EntityOutputStream.toInputStream() that might work for our use case.