Extra data in downloaded files (PHP) - php

Question

I am trying to set up a PHP page to be able to download some log files.
I have tried searching (very hard) for a solution, but I don't really know PHP or HTML, so I have tried many snippets from many sources.
<?php
$dir = "/home/pi/fluidLogs";
$phpfiles = glob("$dir/log*.txt");
foreach ($phpfiles as $phpfile) {
echo '<a href=?file=' . $phpfile . '>' . basename($phpfile) . '</a>';
echo ' - Last mod: ' . date("Ymd-H:i:s", filemtime($phpfile));
echo ' - Size: ' . filesize($phpfile);
echo '</a><br>';
}
if(isset($_GET['file'])){
header('Content-Type: application/octet-stream');
header("Content-Transfer-Encoding: Binary");
header("Content-disposition: attachment; filename=\"" . basename($_GET['file']) . "\"");
readfile($_GET['file']);
}
?>
For example one file should contain,
This is a fake log file.
OK?
with a new line before OK?, but this (and all the downloaded files (via Chrome)) have the same extra data on the first line,
Sorry about the image. The contents were not being displayed correctly.

Answer 1

If user is requesting for a file, i.e. clicked on your anchor link, do not echo the list of files, which are then echoed on top of the file.
<?php
if(isset($_GET['file'])){
header('Content-Type: application/octet-stream');
header("Content-Transfer-Encoding: Binary");
header("Content-disposition: attachment; filename=\"" . basename($_GET['file']) . "\"");
readfile($_GET['file']);
}
else{
$dir = "/home/pi/fluidLogs";
$phpfiles = glob("$dir/log*.txt");
foreach ($phpfiles as $phpfile) {
echo '<a href=?file=' . $phpfile . '>' . basename($phpfile) . '</a>';
echo ' - Last mod: ' . date("Ymd-H:i:s", filemtime($phpfile));
echo ' - Size: ' . filesize($phpfile);
echo '<br>';
}
}
?>