mpart-async icon indicating copy to clipboard operation
mpart-async copied to clipboard

Should be able to handle LF terminated Multipart Boundries

Open e-moran opened this issue 1 year ago • 1 comments

mpart-async currently returns an InvalidBoundry error if a multipart boundary uses the LF line terminator. While the HTTP Spec does call for CRLF terminated Multipart Boundries, it also state that applications should be tolerant where possible. I am currently writing an application that I would like to be tolerant in this regard, but have to work around this in mpart-async by replacing the line endings of the data that I provide to it.

I'm happy to submit a PR to make this change if you agree that it is something that you'd like to support.

e-moran avatar Jun 09 '23 13:06 e-moran

Yes that is definitely something that we should support going forward. Happy for you to write a PR!

I don't know if I would want mixed \n and \r\n within the same request, I.e, it should use either one or the other.

cetra3 avatar Jun 12 '23 01:06 cetra3