Skip to content
GitLab
Closed
Issue created by bmortier@bmortierMaintainer

we should be able to create mail address without having a server and a service referenced

Closed

we should be able to create mail address without having a server and a service referenced

Hello,

lots of people have the need to add email adresses but don't need to manage mail servers. GOsa² and then fd put a strict requirement on having a server and a service to be able to enter mail addresses.

This is wrong and is difficult to understand for those people.

i propose that if the mail plugin is installed and is configured as generic, the default settings, the mail plugin allow to save mail addresses without a server.

This need a change in schema objectclass ( 1.3.6.1.4.1.10098.1.2.1.19.5 NAME 'gosaMailAccount' SUP top AUXILIARY DESC 'Class to mark MailAccounts for GOsa' MUST ( mail $ gosaMailServer $ gosaMailDeliveryMode) MAY ( gosaMailQuota $ gosaMailAlternateAddress $ gosaMailForwardingAddress $ gosaMailMaxSize $ gosaSpamSortLevel $ gosaSpamMailbox $ gosaVacationMessage $ gosaVacationStart $ gosaVacationStop $ gosaSharedFolderTarget))

becoming

objectclass ( 1.3.6.1.4.1.10098.1.2.1.19.5 NAME 'gosaMailAccount' SUP top AUXILIARY DESC 'Class to mark MailAccounts for GOsa' MUST ( mail ) MAY ( gosaMailServer $ gosaMailDeliveryMode $ gosaMailQuota $ gosaMailAlternateAddress $ gosaMailForwardingAddress $ gosaMailMaxSize $ gosaSpamSortLevel $ gosaSpamMailbox $ gosaVacationMessage $ gosaVacationStart $ gosaVacationStop $ gosaSharedFolderTarget))

and i think the same should be done for groups

objectclass ( 1.3.6.1.4.1.38414.10.2.5 NAME 'fdGroupMail' SUP top AUXILIARY DESC 'Basic user group mailing list' MUST ( mail $ gosaMailServer ) MAY ( gosaMailAlternateAddress $ gosaMailForwardingAddress $ gosaMailMaxSize $ fdGroupMailLocalOnly ))

becoming

objectclass ( 1.3.6.1.4.1.38414.10.2.5 NAME 'fdGroupMail' SUP top AUXILIARY DESC 'Basic user group mailing list' MUST ( mail ) MAY ( gosaMailServer $ gosaMailAlternateAddress $ gosaMailForwardingAddress $ gosaMailMaxSize $ fdGroupMailLocalOnly ))

Cheers

(from redmine: issue id 4689, created on 2016-04-11, closed on 2016-04-25)

  • Relations:
  • Changesets:
    • Revision 33c4d0a8 by Benoit MORTIER on 2016-04-14T13:03:36.000Z:
Fixes: #4689 we should be able to create mail address without having a server and a service referenced

removing imap service

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision a5dd0bd5 by Benoit MORTIER on 2016-04-14T13:04:14.000Z:
Fixes: #4689 we should be able to create mail address without having a server and a service referenced

removing imap service

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision fb834b59 by Benoit MORTIER on 2016-04-14T14:39:19.000Z:
Fixes: #4689 we should be able to create mail address without having a server and a service referenced

moved dependancy on other attributes than mail in may

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision dc571388 by Benoit MORTIER on 2016-04-14T14:44:20.000Z:
Fixes: #4689 we should be able to create mail address without having a server and a service referenced

moved dependancy on other attributes than mail in may

Signed-off-by: Mortier Benoit <benoit.mortier@opensides.be>
  • Revision 4e33383f by Côme Chilliet on 2016-04-19T04:09:07.000Z:
Fixes #4689 Changed the way mail method are used

A cache should later be added to mail server list
  • Revision f190199a by Côme Chilliet on 2016-04-19T04:13:59.000Z:
Fixes #4689 Changed the way mail method are used

A cache should later be added to mail server list

Conflicts:
	mail/personal/mail/class_mailAccount.inc
  • Revision 4728555d by Côme Chilliet on 2016-04-19T10:19:15.000Z:
Fixes #4689 Added a cache for mail servers list
  • Revision abcfb5de by Côme Chilliet on 2016-04-19T10:20:25.000Z:
Fixes #4689 Added a cache for mail servers list
  • Revision e3e181a5 by Côme Chilliet on 2016-04-20T08:24:45.000Z:
Fixes #4689 Fixed several errors in last mail method commits
  • Revision a10ea9cc by Côme Chilliet on 2016-04-20T08:25:30.000Z:
Fixes #4689 Fixed several errors in last mail method commits
  • Revision 097b1fa9 by Côme Chilliet on 2016-04-22T08:29:04.000Z:
Fixes #4689 fixed mail method selection and mail debug
  • Revision 68e61b29 by Côme Chilliet on 2016-04-22T08:29:52.000Z:
Fixes #4689 fixed mail method selection and mail debug
  • Revision ff21e00f by Côme Chilliet on 2016-04-22T10:16:21.000Z:
Fixes #4689 Not trying to open mailbox on activation of mail tab
  • Revision 3a999371 by Côme Chilliet on 2016-04-22T10:18:59.000Z:
Fixes #4689 Not trying to open mailbox on activation of mail tab
  • Custom Fields:
    • Bug in version: 1.0.11

    Tasks

    0

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

    Activity

    • bmortier
      bmortier @bmortier ·
      Author Maintainer

      hello,

      removed the basic service

      Cheers

      (from redmine: written on 2016-04-14)

    • bmortier
      bmortier @bmortier ·
      Author Maintainer

      hello,

      moved all server dependancy / quota on may only the mail is on must now

      Cheers

      (from redmine: written on 2016-04-14)

    • Côme Chilliet
      Côme Chilliet @cchilliet ·
      Reporter

      I pushed a first try at changing the way mail method are used, now for each user the mailMethod should be deduced from the selected mail server. If no mail server is selected the «mailMethod» class is used, which does not attempt any server connection.

      We should check that it works with no mail server and with a mail server from cyrus or dovecot plugin. We should also see if we want to disable some fields when no mail server is selected.

      I removed the need for $config->data['SERVERS']['IMAP'] and will remove it from core soon. mailMethod::getMailServers is used instead, for now it does a lookup each time it’s called, once this work we should look into having a cache for this list to avoid useless LDAP lookups. (and reset the cache each time a mail service is updated for instance)

      (from redmine: written on 2016-04-19)

    • Côme Chilliet
      Côme Chilliet @cchilliet ·
      Reporter

      Added a cache for mail servers list

      (from redmine: written on 2016-04-19)

    • Jonathan Swaelens
      Jonathan Swaelens @jswaelens ·
      Developer

      It work when I add a mail tab for a user witout a server (cyrus).

      But when I try with et server (cyrus), I have an error.

      1. Add the cyrus service to a server
      2. Disconnect and reconnect
      3. Go to a user and click on edit
      4. The server dropdown display some information that are not good (:0/notls)
      5. When you select the server, you have this error
       Fatal error: Call to a member function get_ldap_link() on null in /usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-cyrus.inc on line 631

      (from redmine: written on 2016-04-20)

    • Côme Chilliet
      Côme Chilliet @cchilliet ·
      Reporter

      Ok, fixed some problems related to mail server listing and mail methods, please test again.

      (from redmine: written on 2016-04-20)

    • Jonathan Swaelens
      Jonathan Swaelens @jswaelens ·
      Developer

      Now it's good but the sieve connection seem not working.

      (from redmine: written on 2016-04-21)

    • Côme Chilliet
      Côme Chilliet @cchilliet ·
      Reporter

      Should be better now.

      (from redmine: written on 2016-04-22)

    • Côme Chilliet
      Côme Chilliet @cchilliet ·
      Reporter

      And now the error about connexion failed should not be there anymore on first activation.

      (from redmine: written on 2016-04-22)

    • Jonathan Swaelens
      Jonathan Swaelens @jswaelens ·
      Developer

      Close issue

      (from redmine: written on 2016-04-25)

    • Jonathan Swaelens closed

      closed

    • bmortier added Changed label

      added Changed label

    Please register or sign in to reply