Problème sur les liens href dans les mails - solution proposee

DidG 2009-11-02 15:53:32 UTC #1

Bonjour,

ayant constaté que certains liens href contenus dans des emails étaient mal interprétés, j’ai remarqué que la fonction activate_links($content) du fichier phpgwapi/inc/class.html.inc.php traitait ces liens alors qu’elle ne le devrait pas. Il se trouve que lorsqu’un mail contient a href=“http://www.google.fr , la fonction traite la partie http, on se retrouve donc avec un code contenant 2 fois a href= puisque la fonction permet de rendre une URL sous forme d’un lien.
Il me semble que l’erreur vient de la ligne $NotAnchor = '(?<!”|href=|href\s=\s|href=\s|href\s=)’; Le caractère double quote n’est pas interprété comme il faut. Je n’ai pour ma part pas trouvé comme le protéger (si besoin il y a). Je propose par contre l’alternative suivante (3 lignes sont modifiées, elles sont commentées avant leur nouvelle version):
static function activate_links($content) { if (!$content || strlen($content) < 20) return $content; // performance

            // Exclude everything which is already a link
            //$NotAnchor = '(?<!"|href=|href\s=\s|href=\s|href\s=)';
            $NotAnchor = '(?<!href=|href\s=\s|href=\s|href\s=)';

            // spamsaver emailaddress
            $result = preg_replace('/'.$NotAnchor.'mailto:([a-z0-9._-]+)@([a-z0-9_-]+)\.([a-z0-9._-]+)/i',
                    '<a href="#">\\1 AT \\2 DOT \\3</a>',
                    $content);

            //  First match things beginning with http:// (or other protocols)
            //$Protocol = '(http:\/\/|(ftp:\/\/|https:\/\/))';      // only http:// gets removed, other protocolls are shown
            $Protocol = '(http:\/\/|\shttp:\/\/|ftp:\/\/|https:\/\/)';      // only http:// gets removed, other protocolls are shown

            $Domain = '([\w-]+\.[\w-.]+)';
            $Subdir = '([\w\-\.,@?^=%&;:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
            $Expr = '/' . $NotAnchor . $Protocol . $Domain . $Subdir . '/i';

            $result = preg_replace( $Expr, "<a href=\"$0\" target=\"_blank\">$2$3$4</a>", $result );

            //  Now match things beginning with www.
            $NotHTTP = '(?<!:\/\/|" target=\"_blank\">)';   //      avoid running again on http://www links already handled above
            $Domain = 'www(\.[\w-.]+)';
            $Subdir = '([\w\-\.,@?^=%&:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
            $Expr = '/' . $NotAnchor . $NotHTTP . $Domain . $Subdir . '/i';

          //return preg_replace( $Expr, "<a href=\"http://$0\" target=\"_blank\">$0</a>", $result );
            return preg_replace( $Expr, " <a href=\"http://$0\" target=\"_blank\">$0</a>", $result );

    }

Je pense qu’il y a moyen de simplifier en travaillant sur les regex.

En espérant que cela puisse dépanner ceux qui avait le même soucis que moi.

Clermont Communauté - egroupware 1.6.002 - ~600 comptes utilisateurs

tonton_christian 2009-12-12 15:29:01 UTC #2

bonjour,
et merci pour ce retour d’expérience.
jusqu’à la publication de ce message, je ne m’étais pas penché sur le problème pensant que je serais aller à la recherche d’une aiguille dans une botte de foin!

avec ce point d’entrée, j’ai regardé la fonction et je n’arrive pas à la même conclusion. je m’explique.

Cette fonction remplace (ou plus exactement, voudrait remplacer) toute chaine de caractères commençant par “http://” (j’oublie les autres protocoles) ou “www” par la même chaine encadrée par 2 balises [a] en faisant un lien vers la page qu’est censée représenter la chaine.
De plus les caractères http sont supprimés de la chaine affichée.
exemple:
http://www.clcii.fr ==> [a href=“http://www.clcii.fr” target="_blank"]www.clcii.fr[/a]
www.clcii.fr ==> [a href=“http://www.clcii.fr” target=_blank]www.clcii.fr[/a]
http://clcii.fr ==> [a href=“http://clcii.fr” target="_blank"]clcii.fr[/a]

Afin d’éviter de créer un nouveau lien à partir d’une chaine contenue dans une balise [a], les chaines précédées de “href=” ou “”" sont ignorées.

Ca fonctionne correctement tant qu’on a des chaines non précédées de balise [a].

Là ou ça se corse c’est lorsqu’on a des balises [a] qui peuvent contenir des chaines plus compliqées. j’ai trouvé

une variable dans une URL = à http…,
" devant http, remplacé par "

Ce sont 2 cas qui génère une balise [a] à tord.

Donc, je pense que la correction que tu suggère n’est pas la bonne: il faut bien laisser " dans la variable $NotAnchor, et en plus il faut ajouter " et =;

De plus je suggère de commenter la ligne 167; à mons avis, il manque le “;” (autant définir une seule fois la variable qui est déjà définie plus haut)
// $Subdir = ‘([\w-.,@?^=%&:/~+#]*[\w-@?^=%&/~+#])?’;

J’ai fait quelques tests avec ces modifs mais je ne suis pas sûr d’avoir corrigé tous les cas. Je continue mes observation et tiendrai au courant les lecteurs de ce post.

salutations à tous
christian

Didg:

Bonjour,

ayant constaté que certains liens href contenus dans des emails étaient mal interprétés, j’ai remarqué que la fonction activate_links($content) du fichier phpgwapi/inc/class.html.inc.php traitait ces liens alors qu’elle ne le devrait pas. Il se trouve que lorsqu’un mail contient a href=“http://www.google.fr , la fonction traite la partie http, on se retrouve donc avec un code contenant 2 fois a href= puisque la fonction permet de rendre une URL sous forme d’un lien.
Il me semble que l’erreur vient de la ligne $NotAnchor = '(?<!”|href=|href\s=\s|href=\s|href\s=)’; Le caractère double quote n’est pas interprété comme il faut. Je n’ai pour ma part pas trouvé comme le protéger (si besoin il y a). Je propose par contre l’alternative suivante (3 lignes sont modifiées, elles sont commentées avant leur nouvelle version):
static function activate_links($content) { if (!$content || strlen($content) < 20) return $content; // performance
            // Exclude everything which is already a link
            //$NotAnchor = '(?<!"|href=|href\s=\s|href=\s|href\s=)';
            $NotAnchor = '(?<!href=|href\s=\s|href=\s|href\s=)';

            // spamsaver emailaddress
            $result = preg_replace('/'.$NotAnchor.'mailto:([a-z0-9._-]+)@([a-z0-9_-]+)\.([a-z0-9._-]+)/i',
                    '<a href="#">\\1 AT \\2 DOT \\3</a>',
                    $content);

            //  First match things beginning with http:// (or other protocols)
            //$Protocol = '(http:\/\/|(ftp:\/\/|https:\/\/))';      // only http:// gets removed, other protocolls are shown
            $Protocol = '(http:\/\/|\shttp:\/\/|ftp:\/\/|https:\/\/)';      // only http:// gets removed, other protocolls are shown

            $Domain = '([\w-]+\.[\w-.]+)';
            $Subdir = '([\w\-\.,@?^=%&;:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
            $Expr = '/' . $NotAnchor . $Protocol . $Domain . $Subdir . '/i';

            $result = preg_replace( $Expr, "<a href=\"$0\" target=\"_blank\">$2$3$4</a>", $result );

            //  Now match things beginning with www.
            $NotHTTP = '(?<!:\/\/|" target=\"_blank\">)';   //      avoid running again on http://www links already handled above
            $Domain = 'www(\.[\w-.]+)';
            $Subdir = '([\w\-\.,@?^=%&:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
            $Expr = '/' . $NotAnchor . $NotHTTP . $Domain . $Subdir . '/i';

          //return preg_replace( $Expr, "<a href=\"http://$0\" target=\"_blank\">$0</a>", $result );
            return preg_replace( $Expr, " <a href=\"http://$0\" target=\"_blank\">$0</a>", $result );

    }
Je pense qu’il y a moyen de simplifier en travaillant sur les regex.

En espérant que cela puisse dépanner ceux qui avait le même soucis que moi.

Clermont Communauté - egroupware 1.6.002 - ~600 comptes utilisateurs

tonton_christian 2009-12-12 15:39:09 UTC #3

attention, dans mon précedent message, j’avais mis la chaine ‘& quot ;’ qui est interprétée par le navigateur et remplacé par ". (dans ce message, j’insère volontairement un espace après le & et avant le ; pour éviter cette interprétation; il faudra les retirer si vous copiez les corrections à partir du message.

voici donc l’explication corrigée:

Afin d’éviter de créer un nouveau lien à partir d’une chaine contenue dans une balise [a], les chaines précédées de “href=” ou “”" sont ignorées.

Ca fonctionne correctement tant qu’on a des chaines non précédées de balise [a].

Là ou ça se corse c’est lorsqu’on a des balises [a] qui peuvent contenir des chaines plus compliqées. j’ai trouvé

une variable dans une URL = à http…,
" devant http, remplacé par & quot ;

Ce sont 2 cas qui génère une balise [a] à tord.

Donc, je pense que la correction que tu suggère n’est pas la bonne: il faut bien laisser " dans la variable $NotAnchor, et en plus il faut ajouter & quot ; et =;

J’ai fait quelques tests avec ces modifs mais je ne suis pas sûr d’avoir corrigé tous les cas. Je continue mes observation et tiendrai au courant les lecteurs de ce post.

salutations à tous
christian

lastachille 2009-12-15 13:50:01 UTC #4

Bonjour,
voici mon soucis, en utilisant la boite de dialogue de FCKeditor(Version 2.6.2) intégrer à eGroupWare, le lien s’enregistre mais pas comme je le veux, voici quelques exemples (ça sera plus clair) :

/**
*un premier lien en selectionant “glopglop” puis avec le bouton lien et dans la boite de dialogue je rajoute *l’adresse “http://www.glopglop.fr” :
*/

[glopglop ->
http://www.glopglop.fr]
/**
*et moi ce que je veux, c’est simplement “glopglop” qui va pointer vers l’adresse http://www.glopglop.fr
*/

lien en dur : glopglop

J’ai essayer les modifications suggérés, et donc ça ne viens pas de là, si quelqu’un peux m’orienter sur quoi je dois faire les modifications pour ça, ça serait super gentil, merci d’avance !

Configuration: Linux
Firefox 3.5.5

tonton_christian 2009-12-15 14:06:19 UTC #5

ce post parle du traitement des messages reçus qui contiennent, dans le texte, des chaines de caractères http://etc.

toi tu parles de messages écrits; il serait préférable, pour la clarté des échanges, de créer un nouveau sujet.

christian lautier

lastachille:

Bonjour,
voici mon soucis, en utilisant la boite de dialogue de FCKeditor(Version 2.6.2) intégrer à eGroupWare, le lien s’enregistre mais pas comme je le veux, voici quelques exemples (ça sera plus clair) :

/**
*un premier lien en selectionant “glopglop” puis avec le bouton lien et dans la boite de dialogue je rajoute *l’adresse “http://www.glopglop.fr” :
*/

[glopglop ->
http://www.glopglop.fr]
/**
*et moi ce que je veux, c’est simplement “glopglop” qui va pointer vers l’adresse http://www.glopglop.fr
*/

lien en dur : glopglop

J’ai essayer les modifications suggérés, et donc ça ne viens pas de là, si quelqu’un peux m’orienter sur quoi je dois faire les modifications pour ça, ça serait super gentil, merci d’avance !

Configuration: Linux
Firefox 3.5.5