Mail relay to customer server

[kilburn]

First things first: thanks and congratz ispcomm!

Now the annoying part: the patch doesn't apply succesfully against trunk (r1204):

Code:

v2:/usr/local/src/ispcp# svn up
At revision 1204.
v2:/usr/local/src/ispcp# svn st
v2:/usr/local/src/ispcp# patch -p0 < ../mxpatch.txt
patching file engine/ispcp-dmn-mngr
Hunk #1 FAILED at 2079.
Hunk #2 FAILED at 2109.
Hunk #3 FAILED at 2132.
Hunk #4 FAILED at 2202.
Hunk #5 FAILED at 2239.
Hunk #6 succeeded at 3637 with fuzz 2 (offset -24 lines).
5 out of 6 hunks FAILED -- saving rejects to file engine/ispcp-dmn-mngr.rej
patching file gui/include/database-update-functions.php
patching file gui/include/ispcp-functions.php
patching file gui/reseller/edit_domain.php
patching file gui/themes/omega_original/reseller/edit_domain.tpl

The patch itself looks great but I've a few comments:

-) Relay is always set to smtp:[relay_value]. I would suggest leaving the brackets out so the user may set an MX relay (no brackets) or A relay (with brackets).

ispcomm Wrote:The functionality of aliases, pop3 accounts etc is left untouched (but not guaranteed to work when MX is used). It's better to delete all accounts and aliases before using the MX relay.

-) Just don't allow to set relaying if there are any mail accounts and/or aliases for this domain (this is a gui-only check that shouldn't take much effort).

-) Shouldn't we add "reject_unverified_recipients" to the smtpd_recipient_restrictions in the default postfix config?

Edit:

-) When HOSTING_PLANS_LEVEL is set to 'admin' domain edition is made through 'gui/admin/edit_domain.php' (and it's template 'gui/templates/omega_original/admin/edit_domain.tpl') so this files should also be modified.