search

thias / puppet-postfix

Puppet Postfix module
Other
17 stars 117 forks source link

support more params #24

Closed fraenki closed 10 years ago

fraenki commented 10 years ago

add support for $transport_maps, $canonical_maps, $relocated_maps

thias commented 10 years ago

Why not. Did you see that you could already have them with the current code, by passing : 

class { '::postfix::server':
  extra_main_parameters => {
    transport_maps => 'foo',
    canonical_maps => 'bar',
    relocated_maps => 'baz',
  },
}

I'm not against adding more of the most typical configuration directives as their own parameters, but there are a lot, and that's why I added the extra_main_parameters.

fraenki commented 10 years ago

To be honest, I wasn't aware of the extra_main_parameters hash. Personally I'd prefer adding more params over the special hash, though.

thias commented 10 years ago

Included in 0.3.2 and released to the forge.