[SOLUCIONADO] Crear archivo comprimido

[Neoxs]

Hola amigos de Foros:

Estoy armando una página en PHP que arma un archivo ZIP con un par de fotos que hay en el servidor y luego las descarga.
El problema que estoy teniendo es que cuando quiero abrir el archivo descargado, WinZip tira un error de formato desconocido.
El código que estoy usando es:

Código PHP:

  $zip = new ZipArchive();
$filename="fotos.zip";
if($zip -> open($filename, ZIPARCHIVE :: CREATE)==true) {
        $zip->addfile('a.jpg');
                $zip->addfile('b.jpg');
        $zip->close();
    echo "Archivo creado";
    header('Content-Type: application/force-download');
   header('Content-Disposition: attachment; filename='.$filename);
   header('Content-Transfer-Encoding: binary');
   header('Content-Length: '.filesize($filename));
 
    readfile($filename);
} 
   

Alguien sabe qué me está faltando o en qué le estoy errando?
Desde ya muchas gracias
Saludos

[pateketrueke]

¿Lo que muestras es el 100% del código que ejecutas?

[Neoxs]

Sí, ese es todo el código que uso.
El archivo lo arma, pero cuando lo descargo, siempre pesa lo mismo (por más que cambie el código para incluir más imágenes), y cuando lo abro tira un error el WinZip y muestra como si el zip estuviera vacío.
Ahora, si descargo el archivo zip poniendo la URL directamente en el navegador, lo puedo abrir sin problemas! Quiere decir que el código para armar el archivo está funcionando, el problema está en la descarga del archivo.
Será algo de las etiquetas <header>?

[pateketrueke]

De hecho me pregunto ¿por qué imprimes "Archivo creado" antes de leer el archivo que has creado?

Vamos, que ahí estás combinando lo que imprimes con el contenido del archivo así que no me extrañaría que eso corrompa todo.

¿Si sabes que antes de usar header() no debes imprimir nada verdad?

Deberías consultar el manual al respecto.

[germangutierrez]

haciendo ese echo "archivo creado" en ese lugar no va a enviar los headers que esperas. haciendolo despues de enviar las cabeceras vas a corromper los datos zipeados,

[Neoxs]

Entiendo German, lo ponía simplemente para comprobar que entrara en esa parte del código.
Finalmente logré hacerlo andar, usando estos <headers>
header('Content-Description: File Transfer');
header('Content-Type: application/octet-stream');
header('Content-Disposition: attachment; filename=' . $filename);
header('Content-Transfer-Encoding: binary');
header('Expires: 0');
header('Cache-Control: must-revalidate, post-check=0, pre-check=0');
header('Pragma: public');
Saludos, y gracias por la ayuda