roehling / postsrsd

Postfix Sender Rewriting Scheme daemon
319 stars 38 forks source link

multiple domains must be separated by comma #127

Closed dominicraf closed 1 year ago

dominicraf commented 1 year ago

Great utility. But the supplied /etc/default/postsrsd suggests that multiple domains in SRS_EXCLUDE_DOMAINS= can be separated by space or comma. Separating by space however does not work, postsrsd continues to operate on the specified domains and I see messages like this in the log: postsrsd-systemd-launcher[1131464]: /usr/local/share/postsrsd/postsrsd-systemd-launcher: 15: /etc/default/postsrsd: mydomain.com: not found Separating by comma works fine.

Patch with:

--- /etc/default/postsrsd       2022-10-24 14:21:33.497011580 +0100
+++ /tmp/postsrsd_new       2022-10-24 14:23:03.883919689 +0100
@@ -12 +12 @@
-# the domain itself. Separate multiple domains by space or comma.
+# the domain itself. Separate multiple domains by comma.