Emmanuel
Emmanuel
@alex-eri: what's the best workflow to include the unit test, and some others fixes for this problem on the code ? - Should I fork + PR your repo, or...
I've pushed my proposal on your repo on the branch patch-5. If everyone accept it, you can create a Pull Request for merging into the main repo, and close #2128...
Ha! I see the problem: - with multiple write(), there will be output lines that will not be at the maximum size, or even contains a padding in the middle...
For the replace() stuff, it only touches the encoded part of the stream. So wathever the input content is, it can't be converted with "\n" in the base64 format. So...
The test shows that when one stream reader reads multiple chunks, the produced base64 document is not valid. It means that my previous code doesn't work well. It was my...
Still on my todolist (but not on the top) but I miss time to complete it for now. I'll try to prioritize it for the next weeks.