Add an argument for the first separator character after SRS0 and SRS1. The default is now = instead of +

init/postsrsd.default.in

@@ -13,6 +13,10 @@
 #
 #SRS_EXCLUDE_DOMAINS=.example.com,example.org
 
+# First separator character after SRS0 or SRS1.
+# Can be one of: -+=
+#SRS_SEPARATOR==
+
 # Secret key to sign rewritten addresses.
 # When postsrsd is installed for the first time, a random secret is generated
 # and stored in /etc/postsrsd.secret. For most installations, that's just fine.

postsrsd.c

@@ -214,6 +214,7 @@ static void show_help ()
     "Options:\n"
     "   -s<file>       read secrets from file (required)\n"
     "   -d<domain>     set domain name for rewrite (required)\n"
+    "   -a<char>       set first separator character which can be one of: -=+ (default: =)\n"
     "   -f<port>       set port for the forward SRS lookup (default: 10001)\n"
     "   -r<port>       set port for the reverse SRS lookup (default: 10002)\n"
     "   -p<pidfile>    write process ID to pidfile (default: none)\n"
@@ -239,7 +240,7 @@ int main (int argc, char **argv)
   int opt, timeout = 1800, family = AF_INET;
   int daemonize = FALSE;
   char *forward_service = NULL, *reverse_service = NULL,
-       *user = NULL, *domain = NULL, *chroot_dir = NULL;
+       *user = NULL, *domain = NULL, *chroot_dir = NULL, separator = NULL;
   int forward_sock, reverse_sock;
   char *secret_file = NULL, *pid_file = NULL;
   FILE *pf = NULL, *sf = NULL;
@@ -257,7 +258,7 @@ int main (int argc, char **argv)
   tmp = strrchr(argv[0], '/');
   if (tmp) self = strdup(tmp + 1); else self = strdup(argv[0]);
 
-  while ((opt = getopt(argc, argv, "46d:f:r:s:u:t:p:c:X::Dhev")) != -1) {
+  while ((opt = getopt(argc, argv, "46d:a:f:r:s:u:t:p:c:X::Dhev")) != -1) {
     switch (opt) {
       case '?':
         return EXIT_FAILURE;
@@ -270,6 +271,9 @@ int main (int argc, char **argv)
       case 'd':
         domain = strdup(optarg);
         break;
+      case 'a':
+        separator = (char)*optarg;
+        break;
       case 'f':
         forward_service = strdup(optarg);
         break;
@@ -318,6 +322,8 @@ int main (int argc, char **argv)
       case 'e':
         if ( getenv("SRS_DOMAIN") != NULL )
           domain = strdup(getenv("SRS_DOMAIN"));
+        if ( getenv("SRS_SEPARATOR") != NULL )
+          separator = (char)*getenv("SRS_SEPARATOR");
         if ( getenv("SRS_FORWARD_PORT") != NULL )
           forward_service = strdup(getenv("SRS_FORWARD_PORT"));
         if ( getenv("SRS_REVERSE_PORT") != NULL )
@@ -455,7 +461,9 @@ int main (int argc, char **argv)
       srs_add_secret (srs, secret);
   }
   fclose (sf);
-  srs_set_separator (srs, '+');
+  if (separator && srs_set_separator (srs, separator) != SRS_SUCCESS) {
+    srs_set_separator (srs, '=');
+  }
 
   fds[0].fd = forward_sock;
   fds[0].events = POLLIN;