2

Yo quiero que, al reproducir un video en mi página, un contador suba, para eso la pagina con el video tiene el código:

player.on('play', function(data) {
    if (swap[0]==true) {
        var xhttp = new XMLHttpRequest();
        xhttp.open("GET", "http://localhost/play.php?name=3historasdeterror);
        xhttp.send();
        swap[0]=false;
        console.log("funciona");
    }
});

Y el archivo play.php tiene este:

<?php
$access = mysqli_connect("127.0.0.1","****","****","****");
$access -> query("update visits set play = play+1 where name = '".$_GET['name']."'"); 
$access -> close();
?>

El problema de la inyeccion es otro tema, eso ya se como se resuelve, el asunto aquí es que un troll puede tomar el enlace que figuraría en el codigo fuente cargado http://localhost/play.php?name=3historasdeterror (por supuesto si el dominio no fuese localhost) y la puede introducir directamente en el navegador repetidas veces e inflar mis conteos, ¿cómo puedo hacer para que el play.php distinga de dónde proviene el envío del enlace?

Sospecho que se podría hacer si ubico el archivo fuera de la carpeta pública, pero tampoco sé como hacer que el request envíe el enlace a un archivo fuera de la carpeta pública.

2
  • ¿Por que usas Ajax para esto? Y, para controlar el número de visualizaciones (visitas) es mucho más seguro usar servicios como Google Analytics, que te permite usar una API para registro (en la base de datos) y manejo de visitas reales en cualquier URL de tu sitio web.
    – A. Cedano
    el 20 may. 2021 a las 17:20
  • @A.Cedano simpleza quizá, además creo que hay que depender de google lo menos que se pueda, porque acapara todo. el 20 may. 2021 a las 18:08

1 respuesta 1

2

Tienes dos opciones:

1. Enviar encabezado personalizado

Usa el método .setRequestHeader() para agregar un encabezado personalizado:

player.on('play', function(data) {
    if (swap[0]==true) {
        var xhttp = new XMLHttpRequest();
        xhttp.open("GET", "http://localhost/play.php?name=3historasdeterror");
        xhttp.setRequestHeader('AJAX', 'true');
        xhttp.send();
        swap[0]=false;
        console.log("funciona");
    }
});

En PHP usas getallheaders() para obtener los encabezados y comparar; es probable que se agregue el prefijo HTTP_:

<?php
$headers = getallheaders();
// Asignar a una variable para saber que la petición es por AJAX
$ajax = (isset($headers['AJAX']) || isset($headers['HTTP_AJAX']));
// Analizar resultado
if($ajax) {
    // La petición es por AJAX
} else {
    // Se trata de una petición normal
}

2. Agregar una variable a la URL

Simplemente agregas &ajax=true al final de la URL.

Nota: Luego de leer nuevamente tu pregunta, creo que esta opción no sería tan viable para evitar incremento de contadores.

player.on('play', function(data) {
    if (swap[0]==true) {
        var xhttp = new XMLHttpRequest();
        xhttp.open("GET", "http://localhost/play.php?name=3historasdeterror&ajax=true");
        xhttp.send();
        swap[0]=false;
        console.log("funciona");
    }
});

En PHP solo verificas si existe la variable

<?php
$ajax = (isset($_GET['ajax']));
// Analizar resultado
if($ajax) {
    // La petición es por AJAX
} else {
    // Se trata de una petición normal
}

Ten en cuenta que hay herramientas que permiten a los usuarios simular peticiones AJAX y no hay una forma 100% efectiva de evitar el incremento de contadores, pero al menos puedes lidiar con usuarios de escasos conocimientos del tema.

7
  • No funciona ninguna de tus dos opciones. ¿Supuestamente el codigo de play.php lo debía introducir bajo la primera llave del if? el 20 may. 2021 a las 22:39
  • @ArturoViñasSalazar, por favor edita tu pregunta para poner el código actualizado y explicar a qué te refieres con "no funciona", si aparece algún error, no muestra nada o qué está pasando.
    – Triby
    el 20 may. 2021 a las 22:44
  • @ArturoViñasSalazar, no se puede saber el problema hasta que pongas el nuevo código en tu pregunta.
    – Triby
    el 20 may. 2021 a las 22:48
  • esperame un poco, si con esto no encuentras el problema, te pondre el codigo, pero antes el console.log me daba el mensaje que programé cuando daba el play, ahora desde que añado el xhttp.setRequestHeader('AJAX', 'true'); ya no me sale el mansaje de la consola, o sea que el error está en el comienzo. el 20 may. 2021 a las 22:55
  • @ArturoViñasSalazar, si estás viendo la consola, debió aparecer error, mencionando que no se pueden enviar encabezados hasta que se abra la conexión. Solo coloca el encabezado después de .open() y debe funcionar. De cualquier forma, actualicé la respuesta.
    – Triby
    el 20 may. 2021 a las 23:05

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.