Linting, puppet4

Merge remote-tracking branch 'shared/master'

Include upstream minor fixes, get rid of wheezy workaround

Merge branch 'master' into 'master'

small bugfixes

See merge request shared-puppet-modules-group/postfix!21

small bugfixes

Merge remote-tracking branch 'shared/master'

Remove obsolete ensure parameter for concat::fragment.

Recent versions of the concat module error out when one passes an
ensure parameter.

Merge branch '13-dovecot-stretch' into 'master'

Adujst master.cf template for Debian Stretch: support Dovecot 2.x.

Closes #13

See merge request shared-puppet-modules-group/postfix!17

Merge branch 'config_ensure' into 'master'

implement ensure absent for postfix::config

See merge request shared-puppet-modules-group/postfix!16

cf: on Debian Stretch that ships dovecot 2.x, don't pass obsolete -n, but instead -a "${recipient}" (#13)

Same as commit 3ee64eb, applied to the debian-9 template.

Merge remote-tracking branch 'shared/master' into leap_master

implement ensure absent for postfix::config

This does not have a special case for "non-standard" settings, since I
don't know whether it is needed or not.

Merge branch 'bugfix/13-dovecot-lda-v2' into 'master'

master.cf: on Debian distros that ship dovecot 2.x, don't pass obsolete -n, but …

See merge request !14

Merge branch 'bugfix/12-testing-sid' into 'master'

Add symlink to sid template where this module is looking for it (#12).

See merge request !13

Merge branch 'support_concat_2.2.0' into 'master'

Subscribe to the Concat resource instead of the File one.

See merge request !15

Merge remote-tracking branch 'shared/master' into support_concat_2.2.0

Merge branch 'bugfix/gitlab-3-resync-sid-template' into 'master'

Resync sid template

Closes #3

See merge request !7

Fork sid template for Stretch.

Merge remote-tracking branch 'shared/master' into bugfix/gitlab-3-resync-sid-template

Subscribe to the Concat resource instead of the File one.

This is needed with recent versions of the concat module.

cf: on Debian distros that ship dovecot 2.x, don't pass obsolete -n, but instead -a "${recipient}" (#13).

Add symlink to sid template where this module is looking for it (#12).

Fixes this regression brought by dropping lsb facts:

  Could not find template 'postfix/master.cf.debian-stretch/sid.erb'

[bug] Remove leading tabs from master.cf

adding basic postscreen functionality for debian #2303

[bug] Don't use $operatingsystemmajrelease

This fact is not available under the default debian wheezy
facter version.

[bug] Remove dupl. decl. of postfix package

Merge remote-tracking branch 'shared/master' into leap_master

join case statements in init.pp

[refactor] Dont use case statement in resource def

linted init.pp

Merge remote-tracking branch 'shared/master' into leap_master

[deprec] Use @ in front of erb tags in debian cf

Puppet 3 now throws a deprecation warning about this.

make default value for alias_maps optional

make default value for alias_maps optional

documented facter version issue for wheezy hosts in README.md

renamed README -> README.md

Debian sid's cf: merge custom dovecot/deliver options from the Jessie template.

These changes were introduced in other templates in commit 6a44990, let's
propagate them to the sid template.

Resync' the sid cf template with current Debian one.

Updated on 2015-07-18, postfix 2.11.3-1.

use lsbmajdistrelease fact instead of operatingsystemmajrelease because wheezy version of facter doesn't support the latter

Merge branch 'master' into leap

Conflicts:
manifests/satellite.pp

Add GPLv3 license

Merge branch 'fix_satellite_class' into 'master'

Fix satellite class

See merge request !5

Merge branch 'smtp_auth'

small typo in smtp_auth example

lint smtp_auth (spacing style)

fix smtp_auth requirements in comment

last commit moved requirement from the common module to puppetlab's
stdlib

fix postfix::satellite class

i got this error using postfix::satellite:

  Error: Could not retrieve catalog from remote server: Error 400 on
  SERVER: Cannot reassign variable valid_fqdn at
  /etc/puppet/modules/postfix/manifests/satellite.pp:32 …

reassigning a variable within a given scope doesn't work in puppet
see https://docs.puppetlabs.com/puppet/latest/reference/lang_variables.html#no-reassignment

i still think overriding $::fqdn is not a good idea, but for now,
i just want to get this class working again.

use stdlib file_line instead of deprecated line from common module

linted smtp_auth.pp

added smtp_auth.pp

Merge branch 'fix-module_dir-usage' into 'master'

Fix module_dir usage

Change "module_dir" definitions to "common::module_dir".

See merge request !2

Fix usage of the common::module_dir.

Merge branch 'master' into 'master'

Merge with cmaisonnneuve and immerda

This version should be compatible with the cleaned up common module.

See merge request !1

lint templates/master.cf.debian-8.erb

fix module dependency comment in transport and virtual

rename jessie cf template to use version integer

Merge branch 'master' of https://git-ipuppet.immerda.ch/module-postfix

convert to file resources to stdlib's file_line

Add Debian jessie template, is unchanged from earlier version

properly finish the lsb transition and remove very old templates

get rid off lsb facts

cannot reassign variable valid_fqdn (Bug #4125)

Omitting the parameter "valid_fqdn" for postfix::satellite won't
work, even if the machine has a fqdn:

  Cannot reassign variable valid_fqdn at
  /srv/leap/puppet/modules/postfix/manifests/satellite.pp:32

this will fix it, it does:

* make the string in /etc/mailname configurable, defaults to $::fqdn
* postfix::satellite's parameter valid_fqdn is passed to class
postfix's parameter "myorigin" and "mailname"

see https://leap.se/code/issues/4125 for more information.

allow postfix installation to be preseeded (https://leap.se/code/issues/3584)

mention the url to the issue about the services' init script

this way we can follow up on the situation, put pressure on the
community to get this fixed, and remove our hack when it's fixed.

debian's service provider has some issues if the script is absent, we can workaround this problem using the trick of this patch

README: add note about the need to set a global Exec path (#3807)

README: add postfix::disable and fix section headers

setting package correctly to absent

add disable class

Update README to be more fully fleshed out

Syntax fix

Switching to site_postfix convention

Fixing missing delimiter

Switching to parametrized classes

Removing path from postfix::config

Updating header_checks_snippet to use concat

Merge branch 'master' of ssh://labs.riseup.net/shared-postfix

Conflicts:
manifests/definitions/header_checks_snippet.pp
manifests/header_checks.pp
manifests/header_checks_snippet.pp
manifests/init.pp
manifests/tlspolicy.pp
manifests/virtual_regexp_snippet.pp

Move external class inclusion after sanity checks.

Remove useless copy/pasted check.

This define has no $value parameter,
so this check does not make any sense.

Fix wrong define and directory name in documentation.

Use the concat module instead of the concatenated_file defined (Closes: #4074)

Setting postfix_use_submission (2)

Setting postfix_use_submission

Adding postfix_use_smtps parameter

Add path to postfix::config exec command to avoid failure (#3807)

Changing firma path

Initial firma support

Adding mlmmj support (2)

Adding mlmmj support

Adding transport_regexp and virtual_regexp management

put classes and defines in the proper place

To take advantage of puppet's autoloading feature, which will be
mandatory sooner or later. We move all the files in their appropriate
place.

Merge branch 'master' of git://labs.riseup.net/shared-postfix

Use proper variable scope

Facts should be addressed with $::fqdn and so on.

put classes and defines in the proper place

To take advantage of puppet's autoloading feature, which will be
mandatory sooner or later. We move all the files in their appropriate
place.

Making dovecot honor recipient_deliver - wiki.dovecot.org/LDA/Postfix

Add master.cf template for Debian sid.

Imported from postfix 2.8.3-1.

Document the two new variables in the README

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Make 'myorigin' customizable

Not all sites use their fqdn as the 'myorigin' value. The new
$postfix_myorigin variable makes it possible to specify a custom value
for this option.

When no values are specified, keep the default behaviour of using $fqdn
as the value for myorigin.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Parametrize value of inet_interfaces

We don't always want to have postfix listen to all interfaces.

Since the config generally defaults to listening on all interfaces, make
that the default value.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>

Merge branch 'feature/debian_wheezy' of git://gaffer.ptitcanardnoir.org/puppet-module-postfix

Merge branch 'master' of git://gaffer.ptitcanardnoir.org/puppet-module-postfix