__postfix_master interpolates variables #187

New Issue

ungleich-gitea · 2021-11-20T15:21:02Z

ungleich-gitea commented

2021-11-20 15:21:02 +00:00

Created by: telmich

__postfix_master mailman --type unix
--unpriv n --chroot n
--command 'pipe flags=FR user=list argv=/usr/lib/mailman/bin/postfix-to-mailman.py {nexthop} {user}'

results in

cdist:__postfix_master/mailman

mailman unix - n n - - pipe flags=FR user=list argv=/usr/lib/mailman/bin/postfix-to-mailman.py

/cdist:__postfix_master/mailman

-> missing the 2 vars

*Created by: telmich* __postfix_master mailman --type unix \ --unpriv n --chroot n \ --command 'pipe flags=FR user=list argv=/usr/lib/mailman/bin/postfix-to-mailman.py ${nexthop} ${user}' results in # cdist:__postfix_master/mailman mailman unix - n n - - pipe flags=FR user=list argv=/usr/lib/mailman/bin/postfix-to-mailman.py # /cdist:__postfix_master/mailman -> missing the 2 vars

ungleich-gitea added the

Stale

label 2021-11-20 15:21:02 +00:00

ungleich-gitea closed this issue

2021-11-20 15:21:02 +00:00

ungleich-gitea commented

2021-11-20 15:46:55 +00:00

closed

ungleich-gitea commented

2021-11-20 15:46:56 +00:00

Created by: asteven

Nico Schottelius wrote on 11/25/2014 11:26 PM:

@asteven wondering, couldn't / shouldn't we use either __block or __line for backing up __postfix_master? Besides the comment field, which could even be used with __line, I see nothing which could in trigger multiline stuff

__block should work. Don't see how we could use __line here. You either
manage named entries, or you have to parse the whole line/entry and
decide if it has changed or not and so on.

So either, __block, or parse line into each segment and compare is vs
should for each segment.

Cheers,
Steven

*Created by: asteven* Nico Schottelius wrote on 11/25/2014 11:26 PM: > @asteven wondering, couldn't / shouldn't we use either __block or __line for backing up __postfix_master? Besides the comment field, which could even be used with __line, I see nothing which could in trigger multiline stuff __block should work. Don't see how we could use __line here. You either manage named entries, or you have to parse the whole line/entry and decide if it has changed or not and so on. So either, __block, or parse line into each segment and compare is vs should for each segment. Cheers, Steven

ungleich-gitea commented

2021-11-20 15:46:57 +00:00

Created by: telmich

@asteven wondering, couldn't / shouldn't we use either __block or __line for backing up __postfix_master? Besides the comment field, which could even be used with __line, I see nothing which could in trigger multiline stuff

*Created by: telmich* @asteven wondering, couldn't / shouldn't we use either __block or __line for backing up __postfix_master? Besides the comment field, which could even be used with __line, I see nothing which could in trigger multiline stuff

ungleich-gitea commented

2021-11-20 15:46:59 +00:00

Created by: telmich

If I see it correctly, __postfix_master is a re-implementation of __block

*Created by: telmich* If I see it correctly, __postfix_master is a re-implementation of __block

Sign in to join this conversation.