search

andyedinborough / aenetmail

C# POP/IMAP Mail Client
370 stars 153 forks source link

fix headers only with exchange #160

Closed technicallyfeasible closed 10 years ago

technicallyfeasible commented 10 years ago

When synching with Microsoft Exchange then there is sometimes a line within headers consisting only of a space (" "). Using trim will skip those values and leave the stream in a bad state with data remaining.

With headers only reading, the remaining data was not read from the stream, breaking following emails. Added an empty read for those cases just like with mails having a body.

andyedinborough commented 10 years ago

Thanks! Had to remove the strong naming though. ;]

technicallyfeasible commented 10 years ago

Ah, I meant to only add the first checkin, my fault. I should have created a separate branch for that. The second checkin might not have been perfect in terms of interface but I stuck closely to your style so I guess it was ok. It contained the strong naming though, thanks for removing it.

technicallyfeasible commented 10 years ago

unfortunately the second checkin also introduced a bug in the command string. Please have a look at the latest checkin where I have fixed it, sorry for this.