HowtoForge Forums | HowtoForge - Linux Howtos and Tutorials

HowtoForge Forums | HowtoForge - Linux Howtos and Tutorials (http://www.howtoforge.com/forums/index.php)
-   Developers' Forum (http://www.howtoforge.com/forums/forumdisplay.php?f=33)
-   -   Using ISPConfig with Cyrus (or: Cyrus support in ISPConfig) (http://www.howtoforge.com/forums/showthread.php?t=59539)

choongii 13th November 2012 11:18

Using ISPConfig with Cyrus (or: Cyrus support in ISPConfig)
 
Hello,

When using ISPConfig with Cyrus, auto responders and custom rules are not available. However, the options are still visible, which is confusing. I'd like to contribute a patch to resolve this issue, but I am unsure what would be the best solution. I am hoping to receive some feedback on this issue in this thread, so the patch can really contribute something worthwhile which is aligned with the software philosophy of the ISPConfig project.

At the moment I guess there are two possible roads to take:

1) Add checkboxes to enable/disable filters and auto responders per mailserver;

2) Add complete support for cyrus, making it possible to select the Cyrus mailserver as imapd. The aforementioned options can be hidden when Cyrus is selected.

As I'm unsure which path would be the best, and I'm not yet able to oversee the implications of such a change within the project, I'm hoping to receive some feedback from the developers/community so I can define the solution clearly and eventually contribute it.

Thanks for your time,
Choong

till 13th November 2012 11:28

Adding support for a third imap daemon will require more time for testing and maintenance in future, so before we decide if we want to support cyrus imap in ispconfig we should discuss what the benefits over the existing imap implementations are. Why do you want to implement Cyrus support and whyich benefits does cyrus has that are not available in dovecot imap and courier?

choongii 13th November 2012 11:49

Hello Till, thanks for your very quick response!

In my case, we prefer Cyrus because it scales better than either dovecot or courier do, and adheres more strictly to RFC's. But in the end, the reasons for choosing cyrus over dovecot/courier may vary per situation.

I understand the implications such a change would have on future testing/maintenance work, so I guess the big question is: is it worth the (large) effort? We can measure this by the number of people sending in feature requests for cyrus support. I did a search for "cyrus" in the feature request bugtracker, and I get only one result which has two votes, so it doesn't appear to be worth the effort :)

Regardless, we are still able to use cyrus with ISPConfig, but the issue we are having, is that we don't use mail filtering or autoresponders. However, these options cannot be turned off and are always visible to resellers/clients, which results in unnecessary load on our support desk because people get confused.

In light of the above, how would you feel about a patch which makes it possible to turn filtering and autoresponders on/off by using a checkbox per mailserver? We could set them to 'enabled' by default so the current behaviour is not broken. Then we could hide or show the options for filtering and autoresponders depending on these checkboxes.

Thanks again for your time,
Choong

till 13th November 2012 11:59

Quote:

In light of the above, how would you feel about a patch which makes it possible to turn filtering and autoresponders on/off by using a checkbox per mailserver? We could set them to 'enabled' by default so the current behaviour is not broken. Then we could hide or show the options for filtering and autoresponders depending on these checkboxes.
This seems to be a good option in my opinion, the checkbox to enable autoresponder and filtering can be added to e.g. System > server config > mail if it shall be configurable per mail server or in System > Interface config > Mail if it shall be configurable for all servers that belong to this ispconfig installation. I guess the second option might be better. The first option might confuse users as we can not hide the filtering and autoresponder tabs until they saved the new mailbox as we dont know the target mailserver before the address is saved. If more users get interested in cyrus support in future, then we can still develop cyrus support then.

choongii 21st November 2012 16:38

Hi Till,

Thanks again for your reply. The per-installation option does seem the most logical one indeed.

Making it configurable per mailserver could work, if we can somehow (javascript?) dynamically hide tabs depending on the selected domain. We could show or hide the tabs by default, depending on the user's default mailserver. It is more complex than the first option.

Could you let me know, which option would be best for the ISPConfig project? I will then start the development work.

Thanks,
Choong

till 29th November 2012 09:58

I'am sorry for the late reply, totally missed this post.

I would prefer if this option could be set for the whole installation under System > Interface config as this will be the cleaner approach and not per server.

choongii 3rd December 2012 17:48

Hi Till,

Thanks and no worries, I was very late with my response myself :) Thanks for the brainstorming, I will be working on this feature and will have it done around Christmas.

Kind regards,
Choong

choongii 12th January 2013 18:56

1 Attachment(s)
Hi Till,

I've finished up the patch. It adds 3 checkboxes to the interface configuration, to enable the tabs Autoresponder, Mail Filter and Custom Filters respectively. I have two questions at this point:

1. I've tried setting the checkbox to checked by default, but it doesn't appear to work.

2. I'd like to commit the code. What's the best way to do that?

I've attached the patch to this post, it is written against version 3.0.4.5.

Thanks in advance for your time again.

Choong

choongii 28th January 2013 09:18

Hi till,

I was just wondering, if you have had time to look into this yet :)

Thanks in advance,
Choong

till 28th January 2013 19:36

I'am sorry, I missed the post.

The code looks fine, just the following if section seems to be wrong:

if ($global_config['mail']['mailbox_show_custom_rules_tab'] === 'y') {
if($_SESSION["s"]["user"]["typ"] == 'admin') {

I guess it should be:

if($_SESSION["s"]["user"]["typ"] == 'admin' || $global_config['mail']['mailbox_show_custom_rules_tab'] === 'y') {

Quote:

1. I've tried setting the checkbox to checked by default, but it doesn't appear to work.
The defaults for these fields get applied during installation and are stored in the file install/tpl/system.ini

Quote:

2. I'd like to commit the code. What's the best way to do that?
Do you have a svn account for the ispconfig svn server? If not, please send a email to dev [at] ispconfig [dot] org with your forum username and if available your sourceforge.net username and request a svn login.


All times are GMT +2. The time now is 06:41.

Powered by vBulletin® Version 3.8.7
Copyright ©2000 - 2014, vBulletin Solutions, Inc.