Hostname should be included on default value of mydestination

[brauliobo]

default['postfix']['main']['mydestination'] = [node['postfix']['main']['myhostname'], node['hostname'], 'localhost.localdomain', 'localhost'].compact

This makes delivery fails to node['postfix']['main']['myhostname'] and node['hostname'] if the receiver server is at another host.

This also makes override complicated as it can't be done on roles, just directly on node attributes.

I can make a pull request if you agree.

[chasebolt]

@bradenwright can you please provide more detail on this? From the postfix docs it appears to be correct. If you do not want this box to handle mail for itself then you can redefine the attribute in your wrapper. override['postfix']['main']['mydestination'] = ['localhost.localdomain', 'localhost']

"The mydestination parameter specifies what domains this machine will deliver locally, instead of forwarding to another machine. The default is to receive mail for the machine itself."

[bradenwright]

@chasebolt think you may have accidentally included me on this but if I'm wrong then I'd need more info.

Anyways personally I don't see any problems with including it in default or overriding the attribute in recipe/role.

[chasebolt]

Sorry @bradenwright tabbed a bit early and didn't notice the incorrect mention!

@brauliobo