Nodejs banned list

[patilanz]

Hola tengo una funcion que debe de devolver true si un usuario es baneado, pero estoy usando fs para leer la lista de baneos y esta tiene una funcion cellback de modo que no se como devolver true en la funcion de banneo:

Código Javascript:

module.exports={
    isBanned:function(ip){
        var fs=require('fs');
        var fileName='./views/bannedList.txt';
        fs.exists(fileName,function(exists){
            if(exists){
                fs.stat(fileName,function(err,stats){
                    if(err)throw err;
                    fs.open(fileName,'r',function(err,fd){
                        var buffer=new Buffer(stats.size);
                        fs.read(fd,buffer,0,buffer.length,null,function(err,bytesRead,buffer){
                            bannedips=buffer.toString('utf8',0,buffer.length);
                            bannedips=bannedips.split('|||');
                            for(var a=0;a<bannedips.length;a++){
                                console.log(ip);
                                console.log(bannedips[a]);
                                if(bannedips[a]==ip)return true;//Aqui devuelvo true pero en la funcion cellback de fs.read y yo quiero hacer lo en la funcion isBanned
                            }
                            return false;
                        });
                    });
                });
            }else{
                console.log('Banned file don\'t exists. Exit program!');
                process.exit(code=0);
            }
        });
    }
}
 

Alguna idea

[kikeking1]

No termino de entender tu problema amigo, ¿Que te ocurre exactamente? ¿Que no te devuelve el "true/false"?

No he probado el codigo, pero ojeando por encima no veo nada mal en el. Y que el resultado te lo delvuelva al final de todos los callback, es correcto, pues si se devolviera antes, podria ocurrir que el fs aún estubiera leyendo el archivo y no tendrías los valores para comprar.

Asumiendo, que el archivo de los baneado existe (y esta bien referenciado), que se lee y se splitea correctamente y en los consoles log que tienes puesto aparecen los valores que esperas, no debería haber impediento para que se devolviera el "true" si procede invocando a la función.

Concreta un poco más tu duda o tu problema, y podremos ayudarte mejor.

[patilanz]

Iniciado por kikeking1

No termino de entender tu problema amigo, ¿Que te ocurre exactamente? ¿Que no te devuelve el "true/false"?

No he probado el codigo, pero ojeando por encima no veo nada mal en el. Y que el resultado te lo delvuelva al final de todos los callback, es correcto, pues si se devolviera antes, podria ocurrir que el fs aún estubiera leyendo el archivo y no tendrías los valores para comprar.

Asumiendo, que el archivo de los baneado existe (y esta bien referenciado), que se lee y se splitea correctamente y en los consoles log que tienes puesto aparecen los valores que esperas, no debería haber impediento para que se devolviera el "true" si procede invocando a la función.

Concreta un poco más tu duda o tu problema, y podremos ayudarte mejor.

A ver el problema es que todo esta bien pero esto B]if(bannedips[a]==ip)return true;[/B] lo compruebo dentro de la funcion cellback de fs.read(fd,buffer,0,buffer.length,null,function(er r,bytesRead,buffer){
Con lo que si se cumple la condicion anterior la funcion de fs.read devuelve el valor true pero no la funcion de isBanned(); ya que para que la funcion isBanned devuelva true debe de ser algo como esto:

Código Javascript:

module.exports={
    isBanned:function(ip){
        var fs=require('fs');
        var fileName='./views/bannedList.txt';
        fs.exists(fileName,function(exists){
            if(exists){
                fs.stat(fileName,function(err,stats){
                    if(err)throw err;
                    fs.open(fileName,'r',function(err,fd){
                        var buffer=new Buffer(stats.size);
                        fs.read(fd,buffer,0,buffer.length,null,function(err,bytesRead,buffer){
                            bannedips=buffer.toString('utf8',0,buffer.length);
                            bannedips=bannedips.split('|||');
                            for(var a=0;a<bannedips.length;a++){
                                console.log(ip);
                                console.log(bannedips[a]);
                                if(bannedips[a]==ip)return true;//Aqui devuelvo true pero en la funcion cellback de fs.read y yo quiero hacer lo en la funcion isBanned
                            }
                            return false;
                        });
                    });
                });
            }else{
                console.log('Banned file don\'t exists. Exit program!');
                process.exit(code=0);
            }
        });
 
        return true;//>>>Aqui devuelve true la funcion isBanned y no la funcion cellback de fs.read
    }
}
 

Ahora me he explicado bien ??

Saludos

[utan]

Hola,

pero si vas a usar

Código PHP:

  module.exports={
    isBanned:function(ip , callback ){
        var fs=require('fs');
        var fileName='./views/bannedList.txt';
        fs.exists(fileName,function(exists){
            if(exists){
                fs.stat(fileName,function(err,stats){
                    if(err)throw err;
                    fs.open(fileName,'r',function(err,fd){
                        var buffer=new Buffer(stats.size);
                        fs.read(fd,buffer,0,buffer.length,null,function(err,bytesRead,buffer){
                            bannedips=buffer.toString('utf8',0,buffer.length);
                            bannedips=bannedips.split('|||');
                            for(var a=0;a<bannedips.length;a++){
                                console.log(ip);
                                console.log(bannedips[a]);
                                if(bannedips[a]==ip)
                                callback(true); // estas en una funcion anonima no puedes usar return porque estas asiendo una funcion asycronica.
                            }
                           callback(false);
                        });
                    });
                });
            }else{
                console.log('Banned file don\'t exists. Exit program!');
                process.exit(code=0);
            }
        });
 
       callback(true);
    }
} 
   

Digamos que vas a testear un ip, entonce

Código PHP:

  isBanned(192.168.1.1 , function(res){
   // tienes un retorno en el callback
   /**
     *sigue o no con otras funciones o pruebas importantes aquí
     */
   if(res === true){
   }
}) 
   

Ya que tu function que prueba si el ip esta baneado es importante , todo deberia hacerse dentro de la function anónima.

[patilanz]

Hola, muchas gracias funciono :)

Pero una cosa, note que cuando el archivo bannedList.txt es vacio muestra error del buffer.
Como se puede solucionar.

Hay alguna manera de esconder este archivo en nodejs ?? Y cuando un socket se desconecta sale del room automáticamente?? Y si no hay mas sockets el room se elimina?

Saludos

[utan]

Hola,
Yo no eh trabajado con file system en node.js , utilizo el module de mysql , ahora para sacar al usuario del rooms, tienes que usar socket.leave('nombre del cuarto'); y cuando no hay nadie creo que socket.io lo destruye, ojo tomando encuenta que usas socket.io .

Porque te da error si la lista esta vacía?,
es porque el script intenta continuar con valores vacíos?
deberías de probar null y '' en el retorno de tu lista y no continuar con el loop a menos que tenga valores no iguales a espacios o nulos..

espero me explique..