Changeset 109801 in spip-zone for _plugins_/mailsubscribers


Ignore:
Timestamp:
Apr 4, 2018, 1:23:35 PM (3 months ago)
Author:
cedric@…
Message:

Bugfix : quand une adresse etait deja desabonnee, la verification de l'URL echouait car on calculait la cle avec l'email obfusque au lieu d'utiliser celui de l'URL
Du coup maintenant on ne leve plus d'erreur dans les logs et on aboutit sur un message qui dit que l'email n'est pas dans la liste de diffusion

Location:
_plugins_/mailsubscribers/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/mailsubscribers/trunk/inc/mailsubscribers.php

    r108103 r109801  
    3838                }
    3939                if ($arg AND $email) {
    40                         spip_log("mailsubscriber : query_string mal formee, verifiez votre service d'envoi de mails [" . $_SERVER["QUERY_STRING"] . "]",
    41                                 "mailsubscribers" . _LOG_INFO_IMPORTANTE);
     40                        spip_log("mailsubscriber : $email|$arg reconnus malgre la query_string mal formee (verifiez votre service d'envoi de mails) [" . $_SERVER["QUERY_STRING"] . "]", "mailsubscribers" . _LOG_INFO_IMPORTANTE);
    4241                }
    4342        }
    4443        if (!$email OR !$arg){
    45                 spip_log(_request('action')." : (email,arg)=($email,$arg) non trouves", "mailsubscribers"._LOG_ERREUR);
     44                spip_log(_request('action')." : (email,arg)=($email,$arg) non trouves [" . $_SERVER["QUERY_STRING"] . "]", "mailsubscribers"._LOG_ERREUR);
    4645                return false;
    4746        }
     
    5857                $identifiant = "";
    5958                // verifier la cle telle quelle => generique, applicable pour toutes les listes
    60                 $cle = mailsubscriber_cle_action($action, $row['email'], $row['jeton']);
     59                $cle = mailsubscriber_cle_action($action, $email, $row['jeton']);
    6160                if ($arg !== $cle) {
    6261                        $subscriptions = sql_allfetsel('*', 'spip_mailsubscriptions', 'id_mailsubscriber=' . intval($row['id_mailsubscriber']));
    6362                        foreach ($subscriptions as $subscription){
    6463                                // verifier la cle pour cette liste
    65                                 $cle = mailsubscriber_cle_action($action, $row['email'], $row['jeton'] . '+' . $subscription['id_mailsubscribinglist']);
     64                                $cle = mailsubscriber_cle_action($action, $email, $row['jeton'] . '+' . $subscription['id_mailsubscribinglist']);
    6665                                if ($arg == $cle) {
    6766                                        $identifiant = sql_getfetsel('identifiant', 'spip_mailsubscribinglists', 'id_mailsubscribinglist=' . intval($subscription['id_mailsubscribinglist']));
  • _plugins_/mailsubscribers/trunk/paquet.xml

    r109790 r109801  
    22        prefix="mailsubscribers"
    33        categorie="communication"
    4         version="2.7.2"
     4        version="2.7.3"
    55        etat="stable"
    66        compatibilite="[3.0.0;3.2.*]"
Note: See TracChangeset for help on using the changeset viewer.