Skip to content
GitLab
Closed
Issue created by agallavardin@agallavardinReporter

Matching mail suffix between PARTAGE and FD

Closed

Matching mail suffix between PARTAGE and FD

Description

When adding or modifying, partage information related to Sympa Alias, it doesn't work because PARTAGE receive a non-valid request

Distribution Name and Version

Debian Stretch

FusionDirectory Version

ii fusiondirectory 1.3-2~jenkinsbuild28 all Web Based LDAP Administration Program

Plugin with the defect

ii  fusiondirectory-plugin-renater-partage        1.3-2~jenkinsbuild28                         all          Renater partage integration plugin for FusionDirectory
ii  fusiondirectory-plugin-renater-partage-schema 1.3-2~jenkinsbuild28                         all          LDAP schema for FusionDirectory renater partage plugin

PHP version used

ii php7.0 7.0.30-0+deb9u1 all server-side,

Origin of php packages

From deb :

php7.0:
  Installé : 7.0.30-0+deb9u1
  Candidat : 7.0.30-0+deb9u1
 Table de version :
 *** 7.0.30-0+deb9u1 500
        500 http://ftp.fr.debian.org/debian stretch/main amd64 Packages
        500 http://security.debian.org/debian-security stretch/updates/main amd64 Packages
        100 /var/lib/dpkg/status

Steps to Reproduce

  1. add a sympa alias list without partage information
  2. Add partage information
  3. Wait the error

Expected behavior:

Partage information must be created / updated on Partage Infra

Actual behavior:

Error : on browser

Impossible de mettre à jour le compte : Impossible de se connecter à https://api.partage.renater.fr/service/domain/: fopen(https://api.partage.renater.fr/service/domain//CreateGroup/22f97c76cf9a11405cdb5e740fc0aad3): failed to open stream: HTTP request failed! HTTP/1.1 403 Forbidden

with debug :

DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():85) - MAIL: Command: 'CreateGroup'
DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():86) - MAIL: Query: 'name=fakelist%40lists.irstea.fr&description=liste+de+fake&displayName=FAKE+LIST&zimbraMailStatus=enabled&zimbraHideInGal=TRUE&zimbraDistributionListSendShareMessageToNewMembers=FALSE'
DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():142) - MAIL: No Answer:

Reproduces how often: 100%

Additional Information

Basically :

  • our partage domain is : company.fr ( all created object as user, group .. MUST have the mail suffix @company.fr
  • our Sympa domain list is : lists.company.fr

We want to have an alias for 2 reasons

On debug we see : MAIL: Query: 'name=fakelist%40lists.irstea.fr&descri ....

'name=fakelist%40lists.irstea.fr isn"t correct !! It must be name=fakelist%40irstea.fr

the mail domain must be taken from the PARTAGE COS used.

After creation of this group a new request is needed to add as member of fakelist%40irstea.fr the adress contain in CN field : fakelist%40lists.irstea.fr

Edited

    Tasks

    0

    No tasks are currently assigned. Use tasks to break down this issue into smaller parts.

    Activity

    • agallavardin mentioned in issue #5807 (closed)

      mentioned in issue #5807 (closed)

    • bmortier changed the description

      changed the description

    • bmortier changed milestone to %FusionDirectory 1.4

      changed milestone to %FusionDirectory 1.4

    • bmortier added ~759 plugin-renater-partage plugin-sympa labels

      added ~759 plugin-renater-partage plugin-sympa labels

    • bmortier changed milestone to %FusionDirectory 1.3

      changed milestone to %FusionDirectory 1.3

    • bmortier
      bmortier @bmortier ·
      Maintainer

      hello @gallak,

      this seems to me a workaround for you particular situation. Could you share ldap entries of the sympa alias and partage

      i think i understand but i don't want to create error/issues for others users that don't have this specific uses

      Cheers

    • bmortier added 15m of time spent at 2018-08-06

      added 15m of time spent at 2018-08-06

    • bmortier added technical discussion label

      added technical discussion label

    • agallavardin
      agallavardin @agallavardin ·
      Author Reporter

      piece of LDIF :

      objectClass 	sympaAlias
objectClass 	fdRenaterPartageSympaAlias
cn 	dsi.meteo.test@lists.irstea.fr
displayName 	dsi.meteo.test (Liste de distribution SYMPA)
description 	dsi.meteo.test (Liste de distribution SYMPA)
fdRenaterPartageGroupSendShareMessageToNewMembers 	FALSE
sympaServerURL 	https://sympa.irstea.fr
gosaMailServer 	partage.irstea.fr - irstea.fr - staff_irstea.fr
mail 	dsi.meteo.test@lists.irstea.fr
mail 	dsi.meteo.test-owner@lists.irstea.fr
mail 	dsi.meteo.test-editor@lists.irstea.fr
mail 	dsi.meteo.test-unsubscribe@lists.irstea.fr
mail 	dsi.meteo.test-request@lists.irstea.fr
mail 	dsi.meteo.test-subscribe@lists.irstea.fr
fdRenaterPartageGroupMailStatus 	enabled
fdRenaterPartageGroupNotes 	dsi.meteo.test@lists.irstea.fr est une liste de test pour les presta sur https://sympa.irstea.fr
fdRenaterPartageGroupHideInGal

      base on this LDAP entry the request to PARTAGE must forge the name of partage object from cn by :

      • split the cn and take the prefix if this field contain an email address
      • append the Partage domain Name

      I try this quick fix without success ... the partage method send an array pour the name field (why I don't know ..), although that $mail on line 98 contain a string with the correct value for name parameters

      quickfix :

       89   /* We need $mailAccount->cn to return the cn for mailMethod */
 90   public function __get($name)
 91   {
 92     if (($name == 'cn') && isset($this->parent)) {
 93       $mail = $this->parent->getBaseObject()->cn;
 94       if (strpos($mail, '@') === FALSE) {
 95         $mail .= '@'.$this->getEmailDomain();
 96       }else{
 97         /* default rules : we take prefix of cn if cn is a kind of mail address */
 98         $mail = preg_split("/@/",$mail)[0]."@".$this->getEmailDomain();
 99         }
100       return $mail;
101 
102 
103 /* AG      return $this->parent->getBaseObject()->$name; */
104     } elseif (($name == 'mail') && isset($this->parent)) {
105       $mail = $this->parent->getBaseObject()->mail;
106       if (strpos($mail, '@') === FALSE) {
107         $mail .= '@'.$this->getEmailDomain();
108       }
109       return $mail;
110     } else {
111       /* Calling default behaviour */
112       return parent::__get($name);
113     }
114         print_r(" je passe la mais ca dervait pas !!!");
115   }

      error with quickfix

      DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():86) - MAIL: Query: 'name=array&description=dsi.meteo.test+%28Liste+de+distribution+SYMPA%29&displayName=dsi.meteo.test+%28Liste+de+distribution+SYMPA%29&zimbraMailStatus=enabled&zimbraNotes=dsi.meteo.test%40lists.irstea.fr+est+une+liste+de+test+pour+les+presta+sur+https%3A%2F%2Fsympa.irstea.fr%0A&zimbraHideInGal=FALSE&zimbraDistributionListSendShareMessageToNewMembers=FALSE'

    • Côme Chilliet created branch 5848-matching-mail-suffix-between-partage-and-fd

      created branch 5848-matching-mail-suffix-between-partage-and-fd

    • Côme Chilliet mentioned in commit 700ce126

      mentioned in commit 700ce126

    • Côme Chilliet added 30m of time spent at 2018-08-07

      added 30m of time spent at 2018-08-07

    • Côme Chilliet added To Be Tested label

      added To Be Tested label

    • agallavardin
      agallavardin @agallavardin ·
      Author Reporter

      It works perfectly you can close by testing I discovered #5849 (closed)

      cheers

      Edited by agallavardin
    • bmortier added Changed and removed To Be Tested labels

      added Changed and removed To Be Tested labels

    • bmortier removed technical discussion label

      removed technical discussion label

    • bmortier closed

      closed

    Please register or sign in to reply