[PHP] MIME + attachment komt niet goed aan

[mr_wizard]

Ik heb een MIME script geschreven om MIME email met een attachment te kunnen versturen. De attachment moet iedere attachment kunnen zijn.

Als ik nu verstuur komt het allemaal prima aan, filesize in de attachment is OK, maar als ik hem open geeft het desbetreffende programma wat hem opend een foutmelding! Alleen HTM, HTML, TXT en....ZIP komen in goede vorm aan?!

Ik heb encoding op 'base64' en c_type op de file type van de file, die klopt wel. Maar waarom lukt het met ZIP bijvoorbeeld wel? op JPG en EXE crasht hij weer?!?

[SchizoDuckie]

mr_wizard schreef op 16 augustus 2004 @ 11:14:
Ik heb een MIME script geschreven om MIME email met een attachment te kunnen versturen. De attachment moet iedere attachment kunnen zijn.

Als ik nu verstuur komt het allemaal prima aan, filesize in de attachment is OK, maar als ik hem open geeft het desbetreffende programma wat hem opend een foutmelding! Alleen HTM, HTML, TXT en....ZIP komen in goede vorm aan?!

Ik heb encoding op 'base64' en c_type op de file type van de file, die klopt wel. Maar waarom lukt het met ZIP bijvoorbeeld wel? op JPG en EXE crasht hij weer?!?

Sorry, mijn glazen bol is momenteel kapot, dus we kunnen je niet helpen als je geen code samples post

Maar iig, dit is een stukje code uit een PHP E-mail class die ik zelf gebouwd heb:

PHP:

function getAttachments()
    {
        $output = '';
        if (!empty($this->Attachments))
        {
             foreach($this->Attachments as $filename=>$filecontents)
            {
                    $output .= "\n--{$this->outerBoundary}\n";
                    $output .= "Content-Type: application/octetstream;\n\t";
                    $output .= "name=\"{$filename}\"\n";
                    $output .= "Content-Transfer-Encoding: base64\n";
                    $output .= "Content-Disposition: attachment;\n\t";
                    $output .= "filename=\"{$filename}\"\n\n";
                    $output .= chunk_split(base64_encode($filecontents));
                    $output .= "\n\n";
            }
        }
        return ($output);
    }

Hoop dat je daar wat aan hebt.

[ Voor 34% gewijzigd door SchizoDuckie op 16-08-2004 11:20 ]

[Xenon]

Geef je de contenttype correct mee?

[mr_wizard]

De reden dat ik dat niet meestuurde was dat het samen 1500 regels is
hier is een deel:

PHP:

        $target = "./".$HTTP_POST_FILES['attachment']['name'];
        $attach['name'] = $HTTP_POST_FILES['attachment']['name'];
        $attach['type'] = $HTTP_POST_FILES['attachment']['type'];
        copy($HTTP_POST_FILES['attachment']['tmp_name'],$target);
        unlink($HTTP_POST_FILES['attachment']['tmp_name']);
        
        $attachment = $mail->getFile($target);

        function getFile($filename)
        {
                $return = '';
                if ($fp = fopen($filename, 'rb')) {
                        while (!feof($fp)) {
                                $return .= fread($fp, 1024);
                        }
                        fclose($fp);
                        return $return;

                } else {
                        return false;
                }
        }
        function addAttachment($file, $name = '', $c_type, $encoding = 'base64')
        {
                $this->attachments[] = array(
                                                                        'body'                => $file,
                                                                        'name'                => $name,
                                                                        'c_type'        => $c_type,
                                                                        'encoding'        => $encoding
                                                                  );
        }

        $mail->addAttachment($attachment, $attach['name'], $attach['type']);

function send($recipients, $type = 'mail')
        {
                if (!defined('CRLF')) {
                        $this->setCrlf($type == 'mail' ? "\n" : "\r\n");
                }

                if (!$this->is_built) {
                        $this->buildMessage();
                }

                switch ($type) {
                        case 'mail':
                                $subject = '';
                                if (!empty($this->headers['Subject'])) {
                                        $subject = $this->_encodeHeader($this->headers['Subject'], $this->build_params['head_charset']);
                                        unset($this->headers['Subject']);
                                }

                                // Get flat representation of headers
                                foreach ($this->headers as $name => $value) {
                                        $headers[] = $name . ': ' . $this->_encodeHeader($value, $this->build_params['head_charset']);
                                }

                                $to = $this->_encodeHeader(implode(', ', $recipients), $this->build_params['head_charset']);

                                if (!empty($this->return_path)) {
                                        $result = mail($to, $subject, $this->output, implode(CRLF, $headers), '-f' . $this->return_path);
                                } else {
                                        $result = mail($to, $subject, $this->output, implode(CRLF, $headers));
                                }
                                
                                // Reset the subject in case mail is resent
                                if ($subject !== '') {
                                        $this->headers['Subject'] = $subject;
                                }
                                
                                // Return
                                return $result;
                                break;

                        case 'smtp':
                                require_once(dirname(__FILE__) . '/smtp.php');
                                require_once(dirname(__FILE__) . '/RFC822.php');
                                $smtp = &smtp::connect($this->smtp_params);
                                
                                // Parse recipients argument for internet addresses
                                foreach ($recipients as $recipient) {
                                        $addresses = Mail_RFC822::parseAddressList($recipient, $this->smtp_params['helo'], null, false);
                                        foreach ($addresses as $address) {
                                                $smtp_recipients[] = sprintf('%s@%s', $address->mailbox, $address->host);
                                        }
                                }
                                unset($addresses); // These are reused
                                unset($address);   // These are reused

                                // Get flat representation of headers, parsing
                                // Cc and Bcc as we go
                                foreach ($this->headers as $name => $value) {
                                        if ($name == 'Cc' OR $name == 'Bcc') {
                                                $addresses = Mail_RFC822::parseAddressList($value, $this->smtp_params['helo'], null, false);
                                                foreach ($addresses as $address) {
                                                        $smtp_recipients[] = sprintf('%s@%s', $address->mailbox, $address->host);
                                                }
                                        }
                                        if ($name == 'Bcc') {
                                                continue;
                                        }
                                        $headers[] = $name . ': ' . $this->_encodeHeader($value, $this->build_params['head_charset']);
                                }
                                // Add To header based on $recipients argument
                                $headers[] = 'To: ' . $this->_encodeHeader(implode(', ', $recipients), $this->build_params['head_charset']);
                                
                                // Add headers to send_params
                                $send_params['headers']    = $headers;
                                $send_params['recipients'] = array_values(array_unique($smtp_recipients));
                                $send_params['body']       = $this->output;

                                // Setup return path
                                if (isset($this->return_path)) {
                                        $send_params['from'] = $this->return_path;
                                } elseif (!empty($this->headers['From'])) {
                                        $from = Mail_RFC822::parseAddressList($this->headers['From']);
                                        $send_params['from'] = sprintf('%s@%s', $from[0]->mailbox, $from[0]->host);
                                } else {
                                        $send_params['from'] = 'postmaster@' . $this->smtp_params['helo'];
                                }

                                // Send it
                                if (!$smtp->send($send_params)) {
                                        $this->errors = $smtp->errors;
                                        return false;
                                }
                                return true;
                                break;
                }
        }

$result = $mail->send(array('[e-mail]'));

[Xenon]

Hoe bepaal je de waarde van je $c_type ?

misschien tipje

PHP:

$c_type = exec("file -bi ".$file);

[ Voor 7% gewijzigd door Xenon op 16-08-2004 16:14 ]