#202 $CONF['encrypt'] refusing all dovecot methods

v2.3.1
closed-fixed
nobody
Core (82)
5
2014-10-18
2010-07-21
Anonymous
No

Forgotten quotes at line 1207 of functions.inc.php (pacrypt function) lead postfixadmin to die and display the "$CONF[encrypt] = dovecot:md5-crypt will not work because dovecotpw generates a random salt each time. Please use \$CONF[encrypt] = md5crypt instead." message.

Also, shouldn't there be quotes surrounding the "encrypt" in that message ?

Fixed file attached.

Discussion


  • Anonymous
    2010-07-21

     
    Last edit: Anonymous 2014-07-20
    Attachments
  • Thanks for the report!

    Fixed in SVN r849 (2.3 branch), needs to be merged to trunk also.

    Looks like we'll have to do a 2.3.2 release in some weeks ;-)

     
    • status: open --> closed-fixed
     
  • Mickael Nguyen
    Mickael Nguyen
    2010-07-22

    Also, dovecot 2.0 (which is nearing release, see http://dovecot.org/list/dovecot-news/2010-July/000162.html\) will break this setting, because dovecotpw has been removed and its functionality integrated in doveadm.

    So the command to encrypt a password goes from
    dovecotpw -s 'CRYPT-METHOD' to
    doveadm pw -s 'CRYPT-METHOD'

    Maybe something to look into for 2.3.2 :D~

     
  • Oh, nice - I like it when external programs come with incompatible changes :-/ Well, at least it's a new major version ;-)

    $CONF['dovecotpw'] = '/path/to/doveadm -pw'
    could work... (untested, feedback welcome)

    If this works, I'll add a comment to config.inc.php in the 2.3 branch. (I'd like to avoid a bigger change in this branch.)

     
  • Mickael Nguyen
    Mickael Nguyen
    2010-07-23

    It doesn't work.

    After a bit of tinkering, it appears that doveadm does a check on whether it's stdin is a tty or not (It outputs "Fatal: stdin isn't a TTY")

    Guess we can't fix this without "bigger changes". :(