inet_protocols and inet_interfaces added with parenthesis:

Oefenweb / ansible-postfix

Ansible role to set up postfix in Debian-like systems

MIT License

173 stars 82 forks source link

Closed hakkyou closed 5 years ago

hakkyou commented 5 years ago

Input:

  vars:
    postfix_inet_interfaces:
     - 'loopback-only'

Expected behaviour: in /etc/postfix/main.cf:

inet_interfaces = loopback-only
inet_protocols = ipv4

Actual behaviour: in /etc/postfix/main.cf:

inet_interfaces = ['loopback-only']
inet_protocols = ['ipv4']

tersmitten commented 5 years ago

I don't think this is really a bug, inet_interfaces and inet_protocols are both string variable, so you need to provide a string value:

  vars:
    postfix_inet_interfaces: loopback-only

However it's probably better to make both of list type, but that's not backwards compatible :-( Can you you propose a backwards compatible fix?

tersmitten commented 5 years ago