forazar descargas

[oscard41]

Buenas tardes a todos.
Actualmente estoy aprendiendo a forzar descargas en php y bueno tengo el siguiente código:

Código PHP:

<a href='a.php?imagen=mundo.jpg'><img src='./mundo.jpg' style='width:300px;' /></a>
<?php
if(isset($_REQUEST['imagen']))
{
    $name=$_REQUEST['imagen'];
    header("content-disposition:attachment; filename=$name");
    header("content-type:application/octet-stream");
    readfile($name);
}
?>
 

La idea es que al darle click a la imagen se force la descarga y lo hace el problema es que cuando trato de abrir la imagen simplemente no la abre es decir no descarga la imagen...

leí también otros métodos mas seguros para forzar las descargas ya que tengo entendido que con este método al solo modificar la url pueden descargar cualquier otro archivo del servidor...

pero no e probado otros métodos ya que aun no entiendo bien este...

a los que me puedan ayudar de ante mano gracias

[semitasoft]

Te falta agregar un header. que debe estar al inicio.

@header("Content-Type:image/el tipo de imagen");
@header("content-disposition:attachment; filename=$name");
@header("content-type:application/octet-stream");
@readfile($name);

Ten cuidado con este tipo de Códigos. Te pueden bajar todita tu web. Codigos php con tus claves de acceso etc.

La variable recibida a $_REQUEST deberás comprobarla totalmente.
$_REQUEST no es recomendable. usa otra forma de recibir datos.

[Sotelio]

Hola,

Si se toma el código que pegaste tal cual, está mal!

por un lado debes tener

Código HTML:

<a href='a.php?imagen=mundo.jpg'><img src='./mundo.jpg' style='width:300px;' /></a> 

Y en otro archivo, "a.php"

Código PHP:

  if(isset($_REQUEST['imagen']))
{
    $name=$_REQUEST['imagen'];
    header("content-disposition:attachment; filename=$name");
    header("content-type:application/octet-stream");
    readfile($name);
} 
   

Por otro lado, como dicen arriba no es bueno que leas directamente un archivo que viene por la url, sin evaluar lo que se envía, deberías hacer algo como esto:

Código PHP:

  /* indicas los nombre de archivos posibles */
$posibles = array('imagen.jpg', 'otra.jpg', 'etc');
 
/* y luego evaluas */
 
if (isset($_REQUEST['imagen']) && in_array($_REQUEST['imagen'], $posibles)) { 
   

Así te aseguras que el archivo que viene por la url sea sólo archivos "posibles".

Suerte!
@sotelio

[oscard41]

Gracias a todos por sus respuestas

Iniciado por Sotelio

if (isset($_REQUEST['imagen']) && in_array($_REQUEST['imagen'], $posibles)) {

[/php]

Así te aseguras que el archivo que viene por la url sea sólo archivos "posibles".

Suerte!
@sotelio

una pregunta y disculpa mi ignorancia esta parte del codigo que significa?

Código PHP:

in_array($_REQUEST['imagen'], $posibles
 

entiendo que hiciste un array con extenciones de los posibles archivos a descargar, pero esa parte del codigo que hace? repito disculpa mi ignorancia

[Pipe2890]

nbmbnme de error o dejarlo asi

[oscard41]

Iniciado por Pipe2890

nbmbnme de error o dejarlo asi

no lo puedo dejar asi por que me interesa saber que significa!!

las personas que me puedan ayudar con eso Gracias..