Función de seguridad. Mejoras?

[HyQnet]

¡Hola a todos!

Hace tiempo vengo usando esta función que automáticamente limpia $_GET,$_POST,$_COOKIE y $_SESSION

La dejo para que la use el que quiera, y además les pido que el que pueda ayudar de una mano para mejorarla de a poco =)

No creo que algo deje de ser seguro por compartirlo, así que...

Código PHP:

      if(count(@$_GET)>0) 
    { 
        foreach($_GET as $key=>$value) 
        { 
            if(is_array($value)) 
            { 
                $value = ''; 
            } 
             
            $value = strip_tags($value); 
            $value = htmlspecialchars($value); 
            $value = htmlentities($value,ENT_QUOTES,"UTF-8"); 
            $value = trim($value); 
            $value = str_replace("<","&lt;", $value); 
            $value = str_replace(">","&gt;", $value); 
             
            if (get_magic_quotes_gpc()) 
            { 
                $value = stripslashes($value); 
            }    
             
            $_GET[$key]=$value; 
        } 
    } 
 
    if(count(@$_POST)>0) 
    { 
        foreach($_POST as $key=>$value) 
        { 
            if(is_array($value)) 
            { 
                $value = ''; 
            } 
            $value = strip_tags($value); 
            $value = htmlspecialchars($value); 
            $value = htmlentities($value,ENT_QUOTES,"UTF-8"); 
            $value = trim($value); 
            $value = str_replace("<","&lt;", $value); 
            $value = str_replace(">","&gt;", $value); 
             
            if (get_magic_quotes_gpc()) 
            { 
                $value = stripslashes($value); 
            }    
             
            $_POST[$key]=$value; 
        } 
    } 
      
    if(count(@$_COOKIE)>0) 
    { 
        foreach($_COOKIE as $key=>$value) 
        { 
            if(is_array($value)) 
            { 
                $value = ''; 
            }         
            $value = strip_tags($value); 
            $value = htmlspecialchars($value); 
            $value = htmlentities($value,ENT_QUOTES,"UTF-8"); 
            $value = trim($value); 
            $value = str_replace("<","&lt;", $value); 
            $value = str_replace(">","&gt;", $value); 
             
            if (get_magic_quotes_gpc()) 
            { 
                $value = stripslashes($value); 
            }    
             
            $_COOKIE[$key]=$value; 
        } 
    } 
 
    if(count(@$_SESSION)>0) 
    { 
        foreach($_SESSION as $key=>$value) 
        { 
            if(is_array($value)) 
            { 
                $value = ''; 
            }         
            $value = strip_tags($value); 
            $value = htmlspecialchars($value); 
            $value = htmlentities($value,ENT_QUOTES,"UTF-8"); 
            $value = trim($value); 
            $value = str_replace("<","&lt;", $value); 
            $value = str_replace(">","&gt;", $value); 
             
            if (get_magic_quotes_gpc()) 
            { 
                $value = stripslashes($value); 
            }    
             
            $_SESSION[$key]=$value; 
        } 
    }  
   

Problemas actuales de utilizar esta función.

• Los caracteres extraños como á é í etc se transforman en &amp...
• Si usan un editor wysiwyg va a dejar de funciona, ya que el HTML no va a pasar.

Eso es lo que yo encontré hasta ahora... ¿Alguna sugerencia?

Gracias a toda la comunidad =)

[pateketrueke]

Yo no la usaría, hace cosas que no quiero que haga con mis variables, es una solución bastante vieja pues hoy en día hay mejores formas de controlar esto.

[HyQnet]

¡Hola pateketrueke!

La idea de esto es no tener que limpiar variable por variable, sino hacerlo automáticamente...

¿Alguna idea más actual?

Gracias =)

[pateketrueke]

La idea de esto es no tener que limpiar variable por variable, sino hacerlo automáticamente...

Si el problema no es ese, lo entiendo, el problema es todo lo demás, redundancia, replace-hell sin razón, ocultas errores con la arroba, funciones obsoletas, no sé, lleno de malas practicas.

No te ofendas pero tu código es no-bueno.

¿Alguna idea más actual?

Aquí está todo lo que se debe aprender: http://phptherightway.com/

Si deseas compartir a la comunidad hazlo con buenas practicas, se te agradecerá.