I haven't had much time to look into this myself but, from a fairly recent CVS pull, I notice that one's password is echoed when using sieveshell. Perhaps some of the logic in Cyrus::IMAP::authenticate could be used?
I notice the sieveshell script has the following: my $tmpfile = "/tmp/sieveshell.tmp"; Perhaps this should be a bit more careful to avoid possible collisions? Maybe append the $$ or something? This sieveshell uses STARTTLS, right? I think it does since the sieve server won't allow connections otherwise, right? I notice all the perl scripts have something like: #! /bin/sh exec perl -x -S $0 ${1+"$@"} # -*-perl-*- #!perl -w I was wondering, if someone uses the --with-perl configure option, should that setting replace the perl strings in the script header above? In other words, suppose --with-perl=/usr/local/bin/perl is used, perhaps this should be the result? #! /bin/sh exec /usr/local/bin/perl -x -S $0 ${1+"$@"} # -*-perl-*- #!/usr/local/bin/perl -w Lastly, I was wondering if perhaps a '-I' could be put in the header for some of these scripts, especially the cyradm script. We see lots of questions regarding this with each new release. Perhaps a '-I' could be added such that it contained the path of where these Perl modules will eventually reside? -- Amos