[PHP] Header download error

[Verwijderd]

heey tweakers,

Ik ben bezig aan een download systeem, maar ik wil dat ze niet de link kunnen zien waar het bestand zich vind.

Nu heb ik op internet bij (google) gevonden dat je het met dit gedeelte een hoop kan doen.

PHP:

header("Content-Type: application/mp3");

header("Content-Disposition: attachment; filename=$titel2.mp3");

readfile("$waar");

Als ik nu op de link klik dan krijg ik dat download scherm voor me.

Vanaf daar komt het probleem:

Als je op open klik of op opslaan dan kan dat maar hij download maar telkens 352kb

Hij download dus niet het volledige nummer

Weet iemand hoe ik dat kan oplossen ?

het hele script van die file is:

PHP:

session_start();

include("../../includes/config_inc.php");
if($_SESSION['loggedin'] == 12345){

$dj = $_GET['dj'];
$id = $_GET['id'];

    $query = mysql_query("SELECT * FROM tracks WHERE dj = '$dj' AND id = '$id'");
    if($object = mysql_fetch_object($query)){
    $waar = $object->waar;
    $titel = $object->track;
        
        $titel2 = str_replace(" ", "_", $titel);
// set the content as octet-stream
header("Content-Type: application/mp3");

// set it as an attachment and give a file name
header("Content-Disposition: attachment; filename=$titel2.mp3");

// read into the buffer
readfile("$waar");
    
        }
}else{

    header("Location: ../../index.php?module=error&e=2");
    }

[Blaise]

Ik zie niet waar de fout zit die jij zoekt, maar wel een andere. De code is in zijn huidige opzet gevoelig voor SQL injection (regel 6, 7 en 9). Meer informatie daarover: http://en.wikibooks.org/wiki/Programming:PHP:SQL_Injection

En misschien is de MP3 daadwerkelijk 352kb, of is er een half geuploade MP3 gecached?

[Verwijderd]

Blaise schreef op maandag 16 april 2007 @ 18:27:
Ik zie niet waar de fout zit die jij zoekt, maar wel een andere. De code is in zijn huidige opzet gevoelig voor SQL injection (regel 6, 7 en 9). Meer informatie daarover: http://en.wikibooks.org/wiki/Programming:PHP:SQL_Injection

En misschien is de MP3 daadwerkelijk 352kb, of is er een half geuploade MP3 gecached?

Bedankt had ik zelf nie op geled,

Nou de mp3 file is 5.10MB dus neem ik aan dat ie dan wel goed geupload is.

En ik kan het nummer wel gewoon afspelen op de website in een media player ? dus is het nogal een vage fout

[Pete]

Je moet ook even een header toevoegen hoe groot het bestand is

[RAJH]

Probeer eens iets met de Content-Length header. Misschien lost dat je probleem op.

//Edit
Te laat, maar wel met de header erbij

[ Voor 24% gewijzigd door RAJH op 16-04-2007 18:33 ]

[Verwijderd]

PHP:

$query = mysql_query("SELECT * FROM tracks WHERE dj = '".mysql_real_escape_string($_GET['dj'])."' AND id = '".mysql_real_escape_string($_GET['id'])."'");

als ik dit zo neerzet is hij dan zo beveiligd voor mysql injection ?

[BasieP]

een voorbeeldje van iets wat ik gebruik:

PHP:

    header("Expires: Mon, 26 Jul 1997 05:00:00 GMT"); // some day in the past (te maken met caching)
    header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT"); //te maken met caching
    header("Content-type: application/x-download");  //deze en die er onder forceren dat ie gedownload word ipv op de normale manier afgehandeld (wil je waarschijnlijk niet)
    header("Content-type: application/force-download");
    header("Content-Disposition: attachment; filename=\"$fname\""); //de naam (verplicht)
    header("Content-Length: ".@filesize($f)); //de size (semi verplicht)
    header("Content-Transfer-Encoding: binary");  //de encoding (semi verplicht)
    @readfile("$f") or die("File not found.");

Verwijderd schreef op maandag 16 april 2007 @ 18:35:

PHP:

$query = mysql_query("SELECT * FROM tracks WHERE dj = '".mysql_real_escape_string($_GET['dj'])."' AND id = '".mysql_real_escape_string($_GET['id'])."'");

als ik dit zo neerzet is hij dan zo beveiligd voor mysql injection ?

yup

[ Voor 19% gewijzigd door BasieP op 16-04-2007 18:37 ]

[Verwijderd]

oke bedankt

maar nu nog steeds dat andere probleem,

ik heb die content-Lenght header er bij gezet, maar nu geeft hij de juiste naam nie meer goed:S

en wat moet ik daar dan neer zetten ?

PHP:

header("Content-Length:" . filesize( $watmoethier? ));

[Verwijderd]

wat moet ik bij die size invullen ?

[RAJH]

De bestandsnaam, dus $titel2.mp3

[ Voor 32% gewijzigd door RAJH op 16-04-2007 18:47 ]

[Blaise]

-

[ Voor 100% gewijzigd door Blaise op 16-04-2007 18:47 . Reden: spuit 11 ]

[Verwijderd]

BasieP schreef op maandag 16 april 2007 @ 18:36:
een voorbeeldje van iets wat ik gebruik:

PHP:

    header("Expires: Mon, 26 Jul 1997 05:00:00 GMT"); // some day in the past (te maken met caching)
    header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT"); //te maken met caching
    header("Content-type: application/x-download");  //deze en die er onder forceren dat ie gedownload word ipv op de normale manier afgehandeld (wil je waarschijnlijk niet)
    header("Content-type: application/force-download");
    header("Content-Disposition: attachment; filename=\"$fname\""); //de naam (verplicht)
    header("Content-Length: ".@filesize($f)); //de size (semi verplicht)
    header("Content-Transfer-Encoding: binary");  //de encoding (semi verplicht)
    @readfile("$f") or die("File not found.");

[...]


yup

Ik heb dat stuk code van jouw gekopieerd

maar hij werkt nog steeds niet:S

dit is de eerste keer dat ik met zoiets werk dus is het nogal lastig

[BasieP]

Verwijderd schreef op maandag 16 april 2007 @ 18:51:
[...]

Ik heb dat stuk code van jouw gekopieerd

maar hij werkt nog steeds niet:S

dit is de eerste keer dat ik met zoiets werk dus is het nogal lastig

zomaar copy paste is ook in slim, je moet het wel snappen natuurlijk.

wat is je output nu? een nette mp3 file die te klein is? of niks? of text? en werkt ie niet in IE EN firefox? of heb je maar in 1 browser getest?

ow en $f in mijn code is het path naar je file. dat had je zelf ook kunnen vinden als je even had opgezocht wat voor parameter de functie filesize() wilde hebben

[ Voor 14% gewijzigd door BasieP op 16-04-2007 20:45 ]

[Verwijderd]

Dat $f wist ik ook wel,

en kkrijg gwn een bestandje gedownload .mp3 die maar 352kb is .. meer niet

[BasieP]

Verwijderd schreef op maandag 16 april 2007 @ 22:35:
Dat $f wist ik ook wel,

en kkrijg gwn een bestandje gedownload .mp3 die maar 352kb is .. meer niet

* BasieP begint dan een beetje te twijfelen of het bestand compleet op de server staat...
Verder zou het kunnen dat de server de verbinding verbreekt na xx seconden, maar dan zou je dat bij alle http downloads van die host moeten hebben.

[TheDane]

Heb je output buffering aan staan? stuur je nog meer data behalve die mp3?

[NMe]

Ik kan me herinneren dat ik eens een vergelijkbaar probleem had toen ik MP3's met speciale tekens erin probeerde te streamen naar de browser. Zitten er toevallig "gekke" tekens zoals quotes of apostrofs in de bestandsnaam? Probeer die dan eens goed te escapen.

[Verwijderd]

alle rare tekens zijn er uit, maar als nog download hij nog maar 352kb

[Voutloos]

Plaats eens alle response headers zoals je die nu meestuurt, dan is het tenminste duidelijk welke en hoe de gegeven adviezen opgevolgd zijn.

[Verwijderd]

header("Content-type: application/mp3");

header("Content-Disposition: attachment; filename=$titel2.mp3");

readfile("$waar");

[DennusB]

Ze je code is tussen [code] tags!

[Aloys]

Probeer eens of een mp3tje van minder als 352kb wel goed werkt?

[Verwijderd]

PHP:

header("Content-type: application/mp3");

header("Content-Disposition: attachment; filename=$titel2.mp3");

readfile("$waar");

[DennusB]

Let wel even op mensen, ik ken deze gozer!:)
En ik heb de hele code gezien.
$waar = bijvoorbeeld : http://www.desitehier.nl/download/mapje/hierartiest - titel.mp3
Dus met http ervoor! Kan dat wel?

[Voutloos]

OK, dus je doet dus niets met het advies mbt de Content-Length header...

[kunnen]

Timeout na 30 sec?

[Verwijderd]

Dat heb ik wel gedaan, maar dat werkte ook niet egt

want dan ging hij de .php file downloaden.

waar de download code in staat

[ Voor 14% gewijzigd door Verwijderd op 17-04-2007 19:45 ]

[DennusB]

Pulsher schreef op dinsdag 17 april 2007 @ 19:45:
Timeout na 30 sec?

Wat bedoel je daarmee?
Hij download maar een seconde of 2, en kapt er dan mee!

[Voutloos]

@pinda2k6: Nofi, maar dat is gewoon niet waar. Je moet gewoon die header zetten, met de filesize() / strlen() van het te versturen bestand en dan daarna dus gewoon dat bestand outputten.

[ Voor 3% gewijzigd door Voutloos op 17-04-2007 19:48 ]

[Verwijderd]

heb je een voorbeeld ?

[djexplo]

PHP:

function parsefile($fileName, $mp3dir)
{
    $fileString = $mp3dir."/".$fileName;

    if (strstr($_SERVER['HTTP_USER_AGENT'], "MSIE"))
    {
        $fileName = preg_replace('/\./', '%2e', $fileName, substr_count($fileName, '.') - 1);
    }

    if(!$fdl=@fopen($fileString,'r'))
    {
        die("Cannot Open File!");
    }
    else 
    {
        header("Cache-Control: ");// leave blank to avoid IE errors
        header("Pragma: ");// leave blank to avoid IE errors
        header("Content-type: application/octet-stream");
        header("Content-Disposition: attachment; filename=\"".$fileName."\"");
        header("Content-length:".(string)(filesize($fileString)));
        sleep(1);
        fpassthru($fdl);
    }
    return true;
}

$err = parsefile("liedje.mp3" , "documents");

Code die ik zelf ooit geschreven heb, vooral voor IE6 heb ik verschillende codelijntjes toegevoegd ...

[ Voor 10% gewijzigd door djexplo op 17-04-2007 19:59 ]

[Pete]

dennus2006 schreef op dinsdag 17 april 2007 @ 19:42:
Let wel even op mensen, ik ken deze gozer!:)
En ik heb de hele code gezien.
$waar = bijvoorbeeld : http://www.desitehier.nl/download/mapje/hierartiest - titel.mp3
Dus met http ervoor! Kan dat wel?

Daar kan de fout ook in zitten. Dit zou betekenen dat ipv de mp3 gewoon een text-file met errors weergegeven wordt. En ja, het zou goed kunnen dat die altijd dezelfde grootte heeft (alhoewel ik dan 352 kb erg veel vind). Je kunt dit controleren door de gedownloade mp3 even te openen in een texteditor om te zien wat de output is.

[Voutloos]

Verwijderd schreef op dinsdag 17 april 2007 @ 19:50:
heb je een voorbeeld ?

Het voorbeeld van die header was al gegeven, het enige dat je hoeft te doen is de eerste en enige parameter voor de filesize() functie kloppend te maken.

[DennusB]

Dit komt uit een file in wordpad:

code:

<br />
<b>Warning</b>:  filesize() [<a href='function.filesize'>function.filesize</a>]: Stat failed for Make_your_mother_proud (errno=2 - No such file or directory) in <b>/home/domains//public_html/modules/downloaden/downloaden.php</b> on line <b>19</b><br />
<br />
<b>Warning</b>:  Cannot modify header information - headers already sent by (output started at /home/jumpbeatz/domains/jumpbeatz.nl/public_html/modules/downloaden/downloaden.php:19) in <b>/home/domains/jumpbeatz.nl/public_html/modules/downloaden/downloaden.php</b> on line <b>19</b><br />
<br />
<b>Warning</b>:  Cannot modify header information - headers already sent by (output started at /home/jumpbeatz/domains/j/public_html/modules/downloaden/downloaden.php:19) in <b>/home//domains/jumpbeatz.nl/public_html/modules/downloaden/downloaden.php</b> on line <b>21</b><br />
<br />
<b>Warning</b>:  readfile(http://site.nl/tracks/JD/DJ Fukpus_ Make_your_mother_proud.mp3) [<a href='function.readfile'>function.readfile</a>]: failed to open stream: HTTP request failed! HTTP/1.1 400 Bad Request
 in <b>/home//domains/site.nl/public_html/modules/downloaden/downloaden.php</b> on line <b>23</b><br />

Is dus de http:// fout;)

[Voutloos]

filesize(), filesystem dus. De eerste regel in de documentatie: int filesize ( string $filename ).
$filename != $url.

Readfile() geef je toch ook geen url mee als parameter?

[ Voor 17% gewijzigd door Voutloos op 17-04-2007 20:00 ]

[Verwijderd]

<br />
<b>Warning</b>: readfile(./../../tracks/JD/DJ Fukpus_ Make_your_mother_proud.mp3) [<a href='function.readfile'>function.readfile</a>]: failed to open stream: No such file or directory in <b>/home/jumpbeatz/domains/jumpbeatz.nl/public_html/modules/downloaden/downloaden.php</b> on line <b>23</b><br />


dit is een andere error als ik de de link verander naar map voorbeeld:

./../../mapnaam/$url

[AndriesLouw]

Let op, in de mysql-tabel tracks:
Waar geeft het path aan vanuit de dir waarin het bestand staat (bijv.: "../mp3/jaap.mp3")
Track geeft de filenaam aan zoals de gebruiker hem zal opslaan (bijv.: "Jaap.mp3")

Verder:
Ik neem aan dat de config.inc.php een mysql connectie maakt! Of dat je dit zelf doet!

PHP:

<?php
session_start();

include('../../includes/config_inc.php');
if($_SESSION['loggedin'] == '12345'){
    if(!empty($_GET['dj']) && !empty($_GET['id']) && ctype_digit($_GET['id'])){
        $result = mysql_query("SELECT waar, track FROM tracks WHERE dj = '".htmlentities($_GET['dj'],ENT_QUOTES)."' AND id = ".$_GET['id']." LIMIT 1");
        if(mysql_num_rows($result) == '1'){
            $row = mysql_fetch_array($result);
            
            header("Cache-Control: ");
            header("Pragma: ");
            header("Content-type: application/octet-stream"); 
            
            header('Content-Disposition: attachment; filename="'.$row['track'].'"'); 
            header('Content-Length: '.@filesize($row['waar']));
            if(!readfile($row['waar'])){
                echo 'Bestand kon niet gevonden worden, let erop dat "waar":<br />'.PHP_EOL;
                echo '- Niet begint met http:// maar het path aangeeft vanuit de folder van dit script (bijv.: ../mp3/filenaam.mp3)<br />'.PHP_EOL;
            }
        }else{
            echo 'Er is een fout ontstaan bij het uitvoeren van de mysql-query:<br />'.PHP_EOL;
            echo mysql_error().PHP_EOL;
        }
    }else{
        echo 'Het id of de dj is leeggelaten of verkeerd opgegeven (id moet een nummer zijn)';
    }
}else{
    echo 'De $_SESSION[loggedin] bevat niet de value 12345';
}
?>

[ Voor 7% gewijzigd door AndriesLouw op 17-04-2007 20:06 ]

[Creepy]

Verwijderd schreef op dinsdag 17 april 2007 @ 20:01:
<br />
<b>Warning</b>: readfile(./../../tracks/JD/DJ Fukpus_ Make_your_mother_proud.mp3) [<a href='function.readfile'>function.readfile</a>]: failed to open stream: No such file or directory in <b>/home/jumpbeatz/domains/jumpbeatz.nl/public_html/modules/downloaden/downloaden.php</b> on line <b>23</b><br />


dit is een andere error als ik de de link verander naar map voorbeeld:

./../../mapnaam/$url

En wat heb je dan zelf al geprobeerd om die error op te lossen? Niks? Het is hier een forum en geen helpdesk, we verwachten dan ook van je dat je zelf ook wat hebt geprobeerd. Alleen een foutmelding dumpen en hopen dat wij de problemen voor je gaan fixen is niet de bedoeling.

[Verwijderd]

bedankt iedereen

het downloaden werkt nu

oplossing:

readfile aangepast en in de database de url aangepast:D

[djexplo]

Verwijderd schreef op dinsdag 17 april 2007 @ 20:10:
bedankt iedereen

het downloaden werkt nu

oplossing:

readfile aangepast en in de database de url aangepast:D

Tip, test je oplossing onder verschillende browsers en bestandsnamen met commas etc... Want dat het in je huidige browser werkt wil nog niets zeggen...

[Verwijderd]

ik heb het getest in meerdere browsers

maar nu heb ik nog 1 probleempje,

hij geeft niet aan wat het bestands grootte is

[Voutloos]

Content length, content length, content length. De client kan anders gewoon niet weten wat er allemaal nog aan zit te komen na het ontvangen van het 1e pakketje.

[Verwijderd]

ja, maar als ik:

header(' Content-Lenght: ' . filesize ( $url ) ) ;

doe dan werkt het niet want dan zie ik geen bestandsgrote

[Verwijderd]

$waar is bij mij de link naar het bestandje in dit geval /JD/DJ Fukpuss - Make your mother proud.mp3

moet ik die bij die Content-Lenght neer zetten ? of moet daar wat anders ?

[Voutloos]

Vertel nou eens wat die header letterlijk wordt. Geen code, maar gewoon de output.
Staat er helemaal geen getal? Ga dan debuggen wat filesize() met de opgegeven bestandsnaam doet. Sorry voor het bold maken, maar $url is gewoon echt een foute variabele naam.

Overigens is de naam van de header dus Content-Length. Let op de spelling en tevens de header niet beginnen met een spatie.

Verwijderd schreef op dinsdag 17 april 2007 @ 21:25:
$waar is bij mij de link naar het bestandje in dit geval /JD/DJ Fukpuss - Make your mother proud.mp3

moet ik die bij die Content-Lenght neer zetten ? of moet daar wat anders ?

Voutloos schreef op dinsdag 17 april 2007 @ 19:58:
filesize(), filesystem dus. De eerste regel in de documentatie: int filesize ( string $filename ).
$filename != $url.

Readfile() geef je toch ook geen url mee als parameter?

[ Voor 43% gewijzigd door Voutloos op 17-04-2007 21:26 ]

[Verwijderd]

Alles werkt gewoon, de headers werken allemaal

1 ding wat niet werkt is dat hij laat zien hoe groot het bestand is .. ik weet ook niet waar tussen ik hem moet plaatsen ?

want dit is de eerste keer dat ik met zo iets werk

[Verwijderd]

Het is al gelukt:D

beetje zoeken op internet en dan kan je ook je antwoord krijgen

[Swaptor]

Is dat niet gewoon de eerste plek waar je moet kijken?
De mensen voor mij doen dit ook maar voor de fun en zijn zeer zeker niet een persoonlijke helpdesk.

Let verder even op het posten van twee berichten achter elkaar, dit kun je beter oplossen door van de edit-knop () gebruik te maken.

[Verwijderd]

ja dat weet ik

maar kwam ik eigenlijk later op

en ik zoek al 3 dagen op internet maar kwam er niet uit .. en nu had ik net het goeie trefwoord ingevoerd

[Verwijderd]

Heey tweakers,

ik heb jullie hulp nodig ..

al dat werk van de vorige berichten is eigenlijk allemaal niet nodig geweest, ik weet nou wel hoe het zo werkt. Maar: het moet naar een andere server gelinkt worden.

Alleen het probleem, je kan geen http:// gebruiken .. weet iemand hier een oplossing ?

want we willen het wel graag zo doen ?

[OverSoft]

Als je fopen url wrappers gebruikt is het makkelijk (maar dan zou jouw code ook moeten werken).
Indien wrappers aanstaan:
echo file_get_contents($url); // Alleen bij PHP >= 4.4

Indien wrappers uitstaan, kijk dan eens naar de CURL extensie.

Overigens kan je dit OOK op internet vinden, namelijk op PHP.net!

Als je zou eerst eens PHP gaat leren en zelf gaat lezen hoe iets werkt, dan hoef je niet om het uur een vraag te stellen. Zoals al eerder is gesteld, het is hier geen helpdesk.

[ Voor 33% gewijzigd door OverSoft op 18-04-2007 02:23 ]

[NMe]

Verwijderd schreef op dinsdag 17 april 2007 @ 21:42:
beetje zoeken op internet en dan kan je ook je antwoord krijgen

Ik stel voor dat je je voorlopig daar even mee bezig gaat houden. Zo'n beetje elke vraag in dit topic is zo basaal dat je hem zélf, met een beetje moeite, ook had kunnen oplossen. Een deel van de foutmeldingen die je hier postte zijn heel duidelijk; simpelweg wat begrijpend lezen had je in de juiste richting geduwd.

Zo ook je laatste probleem. Alles wat je moet weten over het openen van URLs staat gewoon op de manualpagina die je als het goed is al heel vaak hebt bekeken tijdens dit topic, en deze pagina die van daaruit gelinkt wordt. Ik doe dit topic dan ook op slot; je moet hier prima zo uit kunnen komen.