qzwi/bundlewrap
2
1
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

ldap_scheme_postfix #3

Merged
stillbeben merged 9 commits from ldap_scheme_postfix into main 2022-01-04 09:35:24 +00:00
Conversation 0 Commits 9 Files changed 5 +36
2 changed files with 36 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 90ae486805 - Show all commits

11
bundles/postfix/files/main.cf Normal file
View file

@ -0,0 +1,11 @@
smtpd_banner = $myhostname ESMTP
disable_vrfy_command = yes
smtp_use_tls = yes
smtp_tls_CApath = /etc/pki/tls/certs
smtp_tls_CAfile = /etc/pki/tls/certs/ca-bundle.crt
smtp_tls_mandatory_protocols = !SSLv2,!SSLv3,!TLSv1,!TLSv1.1
smtp_tls_protocols = !SSLv2,!SSLv3,!TLSv1,!TLSv1.1
smtp_tls_security_level = may
smtp_tls_mandatory_ciphers = high

25
bundles/postfix/items.py Normal file
View file

@ -0,0 +1,25 @@
pkg_apt = {
'postfix': {},
stillbeben marked this conversation as resolved
kunsi commented 2022-01-03 11:31:36 +00:00
Review
Copy link

Please install all apt packages via metadata defaults. Bundlewrap can't handle multiple equal item definitions in a repo, so we should stick with only one way of installing packages.

Please install all apt packages via metadata defaults. Bundlewrap can't handle multiple equal item definitions in a repo, so we should stick with only one way of installing packages.
}
svc_systemd = {
'postfix': {
'needs': ['pkg_apt:postfix', 'file:/etc/postfix/main.cf'],
stillbeben marked this conversation as resolved
kunsi commented 2022-01-03 11:34:18 +00:00
Review
Copy link

Please use sets here and below.
Please use one line per item in a set/list, and please add a trailing comma to the last item.

I'd recommend we stick to the PEP8 style guide.

Please use sets here and below. Please use one line per item in a set/list, and please add a trailing comma to the last item. I'd recommend we stick to the [PEP8 style guide](https://www.python.org/dev/peps/pep-0008/#when-to-use-trailing-commas).
}
}
files = {
'/etc/postfix/main.cf': {
'content_type': 'mako',
'needs': ['pkg_apt:postfix'],
'triggers': ['svc_systemd:postfix:restart'],
},
}
directories = {
'/etc/postfix': {
'owner': 'root',
'mode': '1755',
'needs': ['pkg_apt:postfix'],
},
}