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

Reply via email to