content-transfer-encoding check broken

[mgkimsal]

The content-transfer-encoding check is broken - it's only scanning in the rawBodyLines array, but the CTE would be in the headers not the raw body lines. Thus, CTE is never determined - this is hosing up base64 checking on incoming mail.

[mgkimsal]

Ahh... more specifically, this isn't working when an entire message is base64 only, not base64 attachments in the body.

[leoetlino]

Do you mean, we can't check for fully base64 encoded (including the headers) emails using the CTE check/header?

[tomdoo]

Hello
I corrected this issue by replacing this into getBody method because if you do a foreach on the body lines you never find the content type header and you never go to elseif to find content-transfer-encoding.

    //foreach (explode("\n", $this->emailRawContent) as $line) {
    foreach ($this->rawBodyLines as $line) {