isBase64 function problem which added in version 7.0.1

[tigran-m-dev]

Issue Summary

A summary of the issue and the environment in which it occurs. If suitable, include the steps required to reproduce the bug. Please feel free to include screenshots, screencasts, code examples.

Steps to Reproduce

1. Try to send as an attachment text file which contains for example \n
2. Before setContent call convert your attachment as base64

After all this process the isBase64 function will return false, because it not passed ctype_print

Technical details:

This issue in version 7.0.1 and came from merge with number #618

my suggestion to change isBase64 function something like this

private function isBase64($string) {
   $decoded_data = base64_decode($string, true);
   $encoded_data = base64_encode($decoded_data);
   if ($encoded_data != $string) return false;
   else{ 
        $decoded_data=str_ireplace("\t","",$decoded_data);
        $decoded_data=str_ireplace("\n","",$decoded_data);
        $decoded_data=str_ireplace("\r","",$decoded_data);
        if (!ctype_print($decoded_data)) 
            return false;
        }
   
   return true;
}

so new part is this

$decoded_data=str_ireplace("\t","",$decoded_data);
$decoded_data=str_ireplace("\n","",$decoded_data);
$decoded_data=str_ireplace("\r","",$decoded_data);

[tigran-m-dev]

@thinkingserious Could you please check this.

[martijnmelchers]

Why would an attachment only be \n though?

[tigran-m-dev]

@martijnmelchers not only \n. For example if it is contained some formatting text with \n
like this:

some text
some additional text

in this situation ctype_print will return false

[tigran-m-dev]

Just run this simple check

var_dump(ctype_print('some text
some additional text'));

[thinkingserious]

Nice find @developer-devPHP!

Please allow us to send you some swag by filling out this form.

Thanks!