---
title: "Får inte rätt på tvingad nedladdning"
type: "forum-thread"
url: "https://www.webforum.nu/amne/php/190211-får-inte-rätt-på-tvingad-nedladdning"
topic: "PHP"
topic_url: "https://www.webforum.nu/amne/php"
author: "maweer"
published: "2012-08-07T19:01:27.000Z"
updated: "2012-08-10T19:02:28.000Z"
replies: 23
views: 1615
page: 1
pages: 2
language: "sv-SE"
site: "webForum — webforum.nu"
rights: "Upphovsrätten till varje inlägg tillhör dess författare."
attribution: "Citera som: webForum, https://www.webforum.nu/amne/php/190211-får-inte-rätt-på-tvingad-nedladdning"
---

# Får inte rätt på tvingad nedladdning

_Sida 1 av 2._

## #1 — maweer, 2012-08-07T19:01Z

Hur jag än gör med mina headers så laddar den ner rätt men en "tom" fil.

Vad gör jag för fel? 

Här är min kod som anropar get_file.php

```
<?php 

include "conn.php"; 

$sql = mysql_query("SELECT * FROM musik ORDER BY datum DESC LIMIT 50"); 
while ($rad = mysql_fetch_array($sql))

 {
   
  $id = $rad['id'];
  $fil = $rad['fil'];
  $datum = $rad['datum']; 

  echo("<img src=\"images/'.$type.'.gif\">&nbsp;&nbsp;<a href=\"get_file.php/{$rad['fil']}\">Ladda ner</a>");
  echo("&nbsp; - &nbsp;");
  echo($rad['datum']);
  echo("<br>");

  
} 

?>
```

Och här är get_file.php som då är tänkt skall fixa till nedladdningen

```
<?php 
header("Cache-control: private");
header("Content-Type: application/octet-stream");
header("Content-Length: ".filesize("./bilder/".$fil));
header("Content-Disposition: attachment; filename=".$fil);
$fp = fopen("./bilder/".$fil, "rb");
fpassthru($fp);
exit;
?>
```

Permalänk: https://www.webforum.nu/p/190211

## #2 — metalboy, 2012-08-07T19:24Z

Var får $fil sitt värde i get_file.php?

Permalänk: https://www.webforum.nu/p/2300260

## #3 — maweer, 2012-08-07T19:38Z

Det borde väl vara genom $fil = $rad\['fil'\];

Jag har en form som lagrar filerna på en plats och filnamnen i tabellen i DB. 
Sen hämtar jag filen via \<a href=\\"{$rad\['fil'\]} och då blir det ju en helt vanlig länk. 
Men, så tänkte jag att jag använder headers för att tvinga nedladdningen och då la jag till get_file.php

Permalänk: https://www.webforum.nu/p/2300261

## #4 — metalboy, 2012-08-07T19:39Z

Nej, $fil = $rad\['fil'\]; ligger ju i den filen som anropar get_file.php

Permalänk: https://www.webforum.nu/p/2300262

## #5 — maweer, 2012-08-07T19:41Z

Hmmmm....aha!
Då vet jag inte om sanningen skall fram. Jag trodde att det var så!
Den hämtar ju värdet från DB väl...(Hoppas jag)

Permalänk: https://www.webforum.nu/p/2300263

## #6 — metalboy, 2012-08-07T19:50Z

Ok, lova att du sätter dig in i koden för att förstå den. Inte bara klippa och klistra  ;)

nånfil.php

```php
<?php 
include "conn.php"; 

$sql = mysql_query("SELECT * FROM musik ORDER BY datum DESC LIMIT 50"); 
while ($rad = mysql_fetch_array($sql)) {
    $id = $rad['id'];
    $fil = $rad['fil'];
    $datum = $rad['datum']; 

    /* I raden nedan så används $type ... den variabeln verkar inte finnas enligt koden ... men jag lät den vara med i alla fall :D */
    echo '<img src="images/' . $type . '.gif">&nbsp;&nbsp;<a href="get_file.php?file=' . $fil . '">Ladda ner</a>";
    echo "&nbsp; - &nbsp;";
    echo $datum;
    echo "<br>";
} 
?>
```

get_file.php

```php
<?php
$fil = isset($_GET['file']) ? basename($_GET['file']) : false;
if ($fil && file_exists('./bilder/' . $fil)) {
    header("Cache-control: private");
    header("Content-Type: application/octet-stream");
    header("Content-Length: " . filesize("./bilder/".$fil));
    header("Content-Disposition: attachment; filename=" . $fil);
    $fp = fopen("./bilder/" . $fil, "rb");
    fpassthru($fp);
    fclose($fp);
    exit;
} else {
    die('File not found');
}
?>
```

Observera att koden är otestad, men jag tror att den fungerar i alla fall ;)

ÄNDRAT:
Hade råkat skriva $file istället för $fil i get_file.php

Permalänk: https://www.webforum.nu/p/2300264

## #7 — maweer, 2012-08-07T20:06Z

du milde vad du var snabb :)

nånfil.php blev lite konstig tyvärr. Den hämtade inte ut några fil namn och så hämtade den bara ut en fil, så jag provade med den gamla koden, vilket jag tror inte var tanken med det hela faktikst. För jag misstänker väl att det är i min kod i den som felet ligger.

get_file.php
gick den direkt in i else-satsen (file not found)

Permalänk: https://www.webforum.nu/p/2300265

## #8 — metalboy, 2012-08-07T20:07Z

Det enda du behöver ändra i din kod som hämtar info från db är följande

```php
get_file.php/{$rad['fil']}
```

ska vara

```php
get_file.php?file={$rad['fil']}
```

PS. Har du sett att ett litet slarvfel smög sig in i koden för get_file.php? Ändrat nu. DS

Permalänk: https://www.webforum.nu/p/2300266

## #9 — maweer, 2012-08-07T20:20Z

Hahaha ja jösses du är en klippa!!!! :) :) Det funkar sååååå kanon!

Alltså jag har suttit med detta i två dagar och inte fattat ett smack vad som händer!
Hmmmmm...sätta sig in i koden ja :) Hehehe, skall jag vara ärlig....hur gjorde du? :P
Du får jätte gärna förklara för mig :)

.$type. som du frågade om innan är att jag hade tänkt att ha en specifik ikon vid den specifika bilden. Alltså .gif en ikon för det och .jpg en ikon för det.

Tjo....du undrar inte hur glad du gjorde mig precis!
Tack!!!!!!!!!!!!

Permalänk: https://www.webforum.nu/p/2300267

## #10 — maweer, 2012-08-07T20:24Z

Som du märker är jag väldigt nybörjare på detta och det tog mig ett rätt bra tag att bara få rätt på den första koden

Permalänk: https://www.webforum.nu/p/2300268

## #11 — metalboy, 2012-08-08T06:32Z

Bra att du blev nöjd. Angående kodförståelsen, är det något specifikt du vill att jag förklarar?

Permalänk: https://www.webforum.nu/p/2300273

## #12 — maweer, 2012-08-08T07:25Z

Jag blev mer en nöjd...jag blev jätte glad :)

Jo, det var väl i princip hela första raden

```php
$fil = isset($_GET['file']) ? basename($_GET['file']) : false;
```

som jag funderade mest på.

och sen basename() har jag sett många använder men aldrig blivit klok på användningsområdet

I vilket fall som helst så...ett stort tack!

Permalänk: https://www.webforum.nu/p/2300274

## #13 — metalboy, 2012-08-08T08:26Z

```php
$fil = isset($_GET['file']) ? basename($_GET['file']) : false;
```

är ett sätt att skriva en if-else-sats på.
Man hade lika gärna kunna skriva

```php
if (isset($_GET['file'])) {
    $fil = basename($_GET['file']);
} else {
    $fil = false;
}
```

Syntaxen är alltså
*condition* ? *if true* : *if false*

Det nämns lite hastigt i följande manualsida under namnet *ternary conditional operator*:
<http://se.php.net/manual/en/language.expressions.php>

basename() plockar ut själva filnamnsdelen ur en sökväg. Jag la in det i koden för att förhindra att någon ska kunna ladda ner vilken fil som helst från din server. Tänk t.ex. om någon anger get_file.php?file=../index.php, scriptet skulle då låta besökaren ladda ner filen ./bilder/../index.php, dvs din index-fil, och det är nog inte önskvärt.

basename('../index.php') returnerar 'index.php'.

Läs mer om funktionen i manualen på sidan <http://www.php.net/basename>

Manualen är förresten en ypperlig källa till kunskap när man programmerar PHP. Vilken funktion du än är osäker på så skriver du bara adressen <http://www.php.net/funktionsnamn> i adressfältet i din webbläsare så kan du läsa om funktionen, vilka parametrar den tar och vad den returnerar tillsammans med några exempel på hur den används.

Permalänk: https://www.webforum.nu/p/2300275

## #14 — maweer, 2012-08-08T14:10Z

Oj, det var mycket matnyttigt :)
Då är det mao lite farligt att inte använde sig av basename() i sökvägar.

Skall ta till mig av alla tipsen och hoppas att jag kommer ihåg det när jag behöver det igen :)

Permalänk: https://www.webforum.nu/p/2300283

## #15 — metalboy, 2012-08-08T14:22Z

Det som är farligt är att utgå ifrån att bara snällt data kommer in till scriptet.
All data som besökaren på något sätt kan manipulera, oavsett om det är via GET, POST eller COOKIE, så måste du validera att datat är det du förväntar dig. Scriptet blir alltså inte farligt bara för att du inte använder basename(), men du måste kontrollera och tvätta bort sånt som kan vara farligt.

Googla t.ex. på *SQL injection* så kommer du finna ganska mycket som är läsvärt.

Permalänk: https://www.webforum.nu/p/2300284

## #16 — maweer, 2012-08-09T18:29Z

Tack :) Det skall jag verkligen göra! 

Tack för tipset om manualen förresten :)

Har försökt med manualens hjälp och lite snodd kod från andra sidor att få till min variabel $type som är tänkt att visa en specifik bild för gif, en för jpg....etc
Men, det blev inte riktigt som jag hade tänkt mig hahaha

jag tänkte öppna mappen som filerna ligger

```php
$dir = opendir("bilder/");
```

Så här skrevs det i manualen att man var tvungen till att göra med readdir()

```php
while (false !== ($files = readdir($dir))) {
```

Listar alla filer och filtyper

```php
list($files, $type) = explode(" ", $files);
```

Skriver ut bilderna för de olika filtyperna

```php
echo '<img src="images/icon'.$type.'.gif"><br>';
```

Avslutar och stänger

```php
}

closedir($dir);
```

Så långt funkar det (om jag gör det utan min andra kod)....hmmm :r

Så när jag lägga ihop mina två skript:

```php
<?php 
include "conn.php"; 

$sql = mysql_query("SELECT * FROM musik ORDER BY datum DESC LIMIT 50"); 
while ($rad = mysql_fetch_array($sql))
 {
  $id = $rad['id'];
  $fil = $rad['fil'];
  $datum = $rad['datum'];

$dir = opendir("bilder/");
while (false !== ($files = readdir($dir))) {
list($files, $type) = explode(" ", $files);

  echo("<img src=\"images/icon'.$type.'.gif\">&nbsp;&nbsp;<a href=\"get_file.php?file={$rad['fil']}\">Ladda ner</a>");
  echo("&nbsp; - &nbsp;");
  echo($rad['datum']);
  echo("&nbsp; - &nbsp;");
  echo($rad['fil']);
  echo("<br>");
} 
}
closedir($dir);
?>
```

Så listar den ut alla filerna 20 gånger utan bilder.
Jag misstänker att jag inte gjort rätt hahahaha :)

Permalänk: https://www.webforum.nu/p/2300323

## #17 — metalboy, 2012-08-09T18:52Z

Oj, vad exakt är det du vill åstadkomma? :)

Permalänk: https://www.webforum.nu/p/2300325

## #18 — maweer, 2012-08-09T19:20Z

Hahahaha :)
Jo, att man öppnar mappen bilder och läser av vilka filtyper som finns i mappen.

Sen var tanken att den "skriver ut" och visar en ikon för varje post och om det är en gif-fil så visas ikonen icongif.gif, och om det är en jpg-fil så visas ikonen iconjpg.gif.
Där av min konstiga $type

```php
echo("<img src=\"images/icon'.$type.'.gif\">&nbsp;&nbsp;<a href=\"get_file.php?file={$rad['fil']}\">Ladda ner</a>");
```

Permalänk: https://www.webforum.nu/p/2300330

## #19 — metalboy, 2012-08-09T20:14Z

Men filnamnen hämtas väl fortfarande från databasen?
Då behöver du inte öppna någon mapp och läsa innehållet.

Testa följande:

```php
<?php 
include "conn.php"; 

$sql = mysql_query("SELECT * FROM musik ORDER BY datum DESC LIMIT 50"); 
while ($rad = mysql_fetch_array($sql))
 {
  $id = $rad['id'];
  $fil = $rad['fil'];
  $datum = $rad['datum'];
  $type = pathinfo($fil, PATHINFO_EXTENSION);

  echo("<img src=\"images/icon'.$type.'.gif\">&nbsp;&nbsp;<a href=\"get_file.php?file={$rad['fil']}\">Ladda ner</a>");
  echo("&nbsp; - &nbsp;");
  echo($rad['datum']);
  echo("&nbsp; - &nbsp;");
  echo($rad['fil']);
  echo("<br>");
}
?>
```

<http://www.php.net/pathinfo>

Permalänk: https://www.webforum.nu/p/2300331

## #20 — maweer, 2012-08-10T01:48Z

Jo, det har du ju rätt i :)
Aha så du får ut iformationen i mappen med

```php
  $type = pathinfo($fil, PATHINFO_EXTENSION);
```

Coolt....trodde jag behövde all den koden som jag skrev :)
Man kan ju lugnt säga att min kod var lite klumpigare hahaha

Tyvärr får jag inte fram några ikoner, men nu ser ju sidan i alla fall sidan ut som den skall göra igen:)

Permalänk: https://www.webforum.nu/p/2300335

---

Tråden på webben: https://www.webforum.nu/amne/php/190211-får-inte-rätt-på-tvingad-nedladdning  
Nästa sida: https://www.webforum.nu/amne/php/190211-får-inte-rätt-på-tvingad-nedladdning/page2.md