Ticket #1143 (new defect)

Opened 5 years ago

Last modified 7 months ago

liste des abonnés d'une liste depuis le panel

Reported by: anonyme Owned by: squidly
Priority: normal Milestone: alternc-mailman-2.1
Component: alternc-mailman Version: alternc-mailman-1.6
Severity: major Keywords:
Cc:

Description (last modified by fufroma) (diff)

La fonction members de la classe mailman ne tient pas compte de la gestion de domaines virtuels.

Celà induit un fichier vide quand on demande la liste des abonnés, et un message dans l'error.log d'apache-ssl : Liste introuvable : xxx

Il y a 2 modifications à effectuer : dans m_mailman.php et mailman.list

Les 2 fichiers joints sont une proposition de patch.

Dans le patch pour mailman.list, il y à noter que la fonction quotemeta() de perl renvoie les tirets (-) et les points (.) backslashés. Les 2 regexps proposées permettent de les supprimer (sinon mailmain reçoits un truc du genre liste-domaine.com et ne sait pas quoi en faire)

Attachments

mailmain.list.patch Download (328 bytes) - added by anonyme 5 years ago.
patche domaine virtuel pour mailman.list
m_mailman.php.patch Download (361 bytes) - added by anonyme 5 years ago.
patche domaine virtuel pour m_mailman.php

Change History

Changed 5 years ago by anonyme

patche domaine virtuel pour mailman.list

Changed 5 years ago by anonyme

patche domaine virtuel pour m_mailman.php

comment:1 Changed 5 years ago by anonyme

Vu que j'ai posté avec le compte anonyme, si besoin : d(point)rousseau(at)nnx(point)com

comment:2 Changed 5 years ago by anarcat

  • Milestone set to alternc-mailman-1.7

comment:3 Changed 5 years ago by anonyme

Bonjour,

Un petit complément à ce ticket : il serait bon de rajouter les caractères ".", "-" et "_" dans mailman.list, mailman.sub et mailman.unsub comme il a été fait dans mailman.create et mailman.delete.

Et appliquer le patch mailmain.list.patch à mailman.sub et mailman.unsub pour éviter le même genre de problème lié à l'échappement du tiret (je l'ai résolu en local en remplaçant quotemeta par escapeshellarg, mais je ne sais pas si c'est une bonne solution).

Merci !

julien (arobase) nozav (point) org

comment:4 Changed 7 months ago by fufroma

  • Owner set to squidly
  • Description modified (diff)
  • Milestone changed from alternc-mailman-1.7 to alternc-mailman-2.1
Note: See TracTickets for help on using tickets.