|
From: SourceForge.net <no...@so...> - 2010-07-25 19:20:09
|
Bugs item #3032816, was opened at 2010-07-21 23:56 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3032816&group_id=191583 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Core Group: v2.3.1 Status: Closed Resolution: Fixed Priority: 5 Private: No Submitted By: https://www.google.com/accounts () Assigned to: Nobody/Anonymous (nobody) Summary: $CONF['encrypt'] refusing all dovecot methods Initial Comment: 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. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2010-07-25 21:20 Message: "bigger changes" also mean that a separate bugreport makes sense. It's the old game: only one bug per bugreport ;-) I just opened https://sourceforge.net/tracker/?func=detail&aid=3034389&group_id=191583&atid=937964 The new bugreport also contains a code change that you could test - please reply there ;-) ---------------------------------------------------------------------- Comment By: Mickael Nguyen (mickaelnguyen) Date: 2010-07-23 18:35 Message: 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". :( ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2010-07-23 00:05 Message: 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.) ---------------------------------------------------------------------- Comment By: Mickael Nguyen (mickaelnguyen) Date: 2010-07-22 04:20 Message: 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~ ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2010-07-22 01:07 Message: 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 ;-) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3032816&group_id=191583 |
