Can you show which error you got when ppolicy is set? Also I think the openldap26 change (as in the other MR) is misplaced.
- openldap26-server change needs to be clarified
- Rest is fine for merging
Hell @tabulon !
Sorry for the late reply, a lot of things going on atm.
In regards to PRs: 1 would be preferred, including tests (where appropriate).
I just skimmed above comments and I…
Good morning @tabulon !
I think having identical behaviour on both sites makes a lot of sense.
I remember that the motivation for always using /bin/sh on the remote side was that in any case…
Fixed today, applied the missing ' directly. Thanks for reporting.
Wow. we should have caught that one earlier. Will merge it tomorrow.
I suggest to reach out to Dennis, as he mixed {} and () functions, both which technically are correct due to the compound, but he might have had a motivation for potentially keeping x local or so.