search

voxpupuli / puppet-postfix

Puppet postfix module
Apache License 2.0
70 stars 172 forks source link

if relayhost is defined, then use the satellite profile #310

Closed kapouik closed 1 year ago

kapouik commented 2 years ago

Pull Request (PR) description

when relayhost is define in the init.pp, there is nothing call behind it. As it, if you use it like show in the documentation, it don't work. After reading the code, it look like that the standard usage for relayhost is to use satellite profile that permit to configure postfix to send all email to $relayhost.

This Pull Request (PR) fixes the following issues

Fixes #295

kapouik commented 2 years ago

I have made the necessary modification in the documentation and in the error message.

vox-pupuli-tasks[bot] commented 2 years ago

Dear @kapouik, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

smortex commented 1 year ago

It looks like that branch got somewhat messed-up for some times. I will close this PR for now, but feel free to submit a cleaned-up version if you want your change to be upstreamed.