postfixadmin-tracker Mailing List for PostfixAdmin (Page 4)
Brought to you by:
christian_boltz,
gingerdog
You can subscribe to this list here.
2007 |
Jan
|
Feb
|
Mar
|
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
(5) |
Oct
(67) |
Nov
(83) |
Dec
(47) |
---|---|---|---|---|---|---|---|---|---|---|---|---|
2008 |
Jan
(57) |
Feb
(15) |
Mar
(21) |
Apr
(38) |
May
(27) |
Jun
(38) |
Jul
(35) |
Aug
(50) |
Sep
(8) |
Oct
(9) |
Nov
(59) |
Dec
(59) |
2009 |
Jan
(27) |
Feb
(42) |
Mar
(63) |
Apr
(46) |
May
(26) |
Jun
(25) |
Jul
(40) |
Aug
(19) |
Sep
(17) |
Oct
(35) |
Nov
(26) |
Dec
(21) |
2010 |
Jan
(11) |
Feb
(19) |
Mar
(40) |
Apr
(25) |
May
(23) |
Jun
(17) |
Jul
(10) |
Aug
(18) |
Sep
(21) |
Oct
(12) |
Nov
(10) |
Dec
(22) |
2011 |
Jan
(30) |
Feb
(23) |
Mar
(23) |
Apr
(38) |
May
(32) |
Jun
(19) |
Jul
(20) |
Aug
(36) |
Sep
(11) |
Oct
(28) |
Nov
(4) |
Dec
(4) |
2012 |
Jan
(6) |
Feb
(3) |
Mar
(16) |
Apr
(28) |
May
(29) |
Jun
(10) |
Jul
(2) |
Aug
(3) |
Sep
|
Oct
(13) |
Nov
(1) |
Dec
(1) |
2013 |
Jan
(11) |
Feb
(7) |
Mar
(29) |
Apr
(2) |
May
(3) |
Jun
(15) |
Jul
(8) |
Aug
(5) |
Sep
(5) |
Oct
(4) |
Nov
(27) |
Dec
(81) |
2014 |
Jan
(12) |
Feb
(13) |
Mar
(5) |
Apr
|
May
(41) |
Jun
(16) |
Jul
(7) |
Aug
(10) |
Sep
(24) |
Oct
(50) |
Nov
|
Dec
(2) |
2015 |
Jan
(5) |
Feb
(2) |
Mar
(7) |
Apr
(20) |
May
(1) |
Jun
(3) |
Jul
(12) |
Aug
(1) |
Sep
(17) |
Oct
(5) |
Nov
(20) |
Dec
(10) |
2016 |
Jan
(10) |
Feb
(11) |
Mar
(22) |
Apr
(30) |
May
(33) |
Jun
(3) |
Jul
|
Aug
(12) |
Sep
(20) |
Oct
(11) |
Nov
(15) |
Dec
(8) |
2017 |
Jan
(1) |
Feb
(11) |
Mar
(10) |
Apr
|
May
(3) |
Jun
|
Jul
|
Aug
(1) |
Sep
|
Oct
(1) |
Nov
|
Dec
(3) |
2018 |
Jan
(2) |
Feb
|
Mar
(1) |
Apr
|
May
(4) |
Jun
(2) |
Jul
(4) |
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
2019 |
Jan
|
Feb
|
Mar
|
Apr
(3) |
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
2021 |
Jan
|
Feb
|
Mar
(1) |
Apr
|
May
|
Jun
|
Jul
|
Aug
|
Sep
|
Oct
|
Nov
|
Dec
|
From: SourceForge.net <no...@so...> - 2012-06-29 17:08:39
|
Bugs item #3539027, was opened at 2012-06-29 10:08 Message generated for change (Tracker Item Submitted) made by mindcontrolled You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3539027&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.5 Status: Open Resolution: None Priority: 5 Private: No Submitted By: mindcontrolled (mindcontrolled) Assigned to: Nobody/Anonymous (nobody) Summary: Cannot redeclare maildir_name_hook() Initial Comment: This was reported earlier in #3305278 but closed because it could not be reproduced, but I just ran into this so here to see if we can get this solved forever ;-) When this occured, I just downloaded the 2.3.5 tar.gz. Unpacked. Then began editting the config first, going through all the options one by one. I didn't use my previous config file which was from 2.3.3, and I didn't copy any other files over from that version. This 2.3.5 was a clean install. Last thing I changed was maildir_name_hook. I uncommented the function, then attempted to run setup.php for generating a password hash. No error was displayed, but the page stopped rendering right after "Depends on: presence config.inc.php - OK". After the closing </li> there is an empty line in the source, it stops there. In the logs I found a "PHP Fatal error: Cannot redeclare maildir_name_hook() (previously declared in /home/websites/www/postfixadmin/config.inc.php:172) in /home/websites/www/postfixadmin/config.inc.php on line 179". Line 172: The line where function maildir_name_hook starts. Line 179: The line where the function ends by means of a closing curly bracket. If I uncomment the function completely, setup.php runs fine. Uncommenting it stops setup.php from running. Looking at setup.php, line 123 and 126: The first one has a require_once() of the config, but the latter has a require() of the config. So in short, it seems that setup.php is loading config.inc.php twice, once by require_once() plus once by using require(), and the second time PHP will complain and halt on redeclaration of maildir_name_hook(). I'm not sure what the intention of this code is; on line 124 it sets $config_loaded = 1 and then it loads the config again. My initial response would be to remove the require() on line 126. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3539027&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-26 19:01:11
|
Bugs item #3538192, was opened at 2012-06-26 10:22 Message generated for change (Settings changed) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3538192&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: Interface (example) Group: None >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Andrey Volkov (axiff) Assigned to: Nobody/Anonymous (nobody) Summary: No input focus on login page Initial Comment: templates/login.php Typo at line: script tyle="text/javascript" script tyPe ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-06-26 12:01 Message: Good catch - I noticed this a while ago, but overlooked the typo ;-) Fixed in SVN r1404 (2.3 branch). I also copied the JS code to focus the username field to the user login page. SVN trunk already had the correct and typo-free code. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3538192&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-26 17:22:21
|
Bugs item #3538192, was opened at 2012-06-26 10:22 Message generated for change (Tracker Item Submitted) made by axiff You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3538192&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: Interface (example) Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: xiff (axiff) Assigned to: Nobody/Anonymous (nobody) Summary: No input focus on login page Initial Comment: templates/login.php Typo at line: script tyle="text/javascript" script tyPe ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3538192&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 23:18:05
|
Patches item #3536134, was opened at 2012-06-18 16:13 Message generated for change (Comment added) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: making original subject referable in the vacation mail Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- >Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:18 Message: I just added the patch information to our OpenSource project, too: https://confluence.clazzes.org/x/NIAo ---------------------------------------------------------------------- Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:13 Message: Some more information on our patch: This patch was made towards vacation.pl as of debian's postfixadmin 2.3.5-1 package for wheezy (the current testing release). It replaces any '$SUBJECT' (that is, a dollar sign and the word subject in upper case letters) in the vacation mail subject template with the subject of the original e-mail. For example: If the vacation mail subject is set to 'Re: $SUBJECT', the vacation mail gets the same subject as a common reply mail created by the recipient using an e-mail application. To use this as the default subject template, the langauge file has to be changed. This is from our /usr/share/postfixadmin/languages/en.lang file: $PALANG['pUsersVacation_subject_text'] = 'Re: $SUBJECT'; ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 23:13:50
|
Patches item #3536134, was opened at 2012-06-18 16:13 Message generated for change (Comment added) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: making original subject referable in the vacation mail Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- >Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:13 Message: Some more information on our patch: This patch was made towards vacation.pl as of debian's postfixadmin 2.3.5-1 package for wheezy (the current testing release). It replaces any '$SUBJECT' (that is, a dollar sign and the word subject in upper case letters) in the vacation mail subject template with the subject of the original e-mail. For example: If the vacation mail subject is set to 'Re: $SUBJECT', the vacation mail gets the same subject as a common reply mail created by the recipient using an e-mail application. To use this as the default subject template, the langauge file has to be changed. This is from our /usr/share/postfixadmin/languages/en.lang file: $PALANG['pUsersVacation_subject_text'] = 'Re: $SUBJECT'; ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 23:13:26
|
Bugs item #3536132, was opened at 2012-06-18 15:53 Message generated for change (Comment added) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&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: Vacation Group: v2.3.5 >Status: Closed >Resolution: Duplicate Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: original subject should be referable vacation mail, patch Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- >Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:13 Message: Moving it to patches tracker ... ---------------------------------------------------------------------- Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:02 Message: Some more information on our patch: This patch was made towards vacation.pl as of debian's postfixadmin 2.3.5-1 package for wheezy (the current testing release). It replaces any '$SUBJECT' (that is, a dollar sign and the word subject in upper case letters) in the vacation mail subject template with the subject of the original e-mail. For example: If the vacation mail subject is set to 'Re: $SUBJECT', the vacation mail gets the same subject as a common reply mail created by the recipient using an e-mail application. To use this as the default subject template, the langauge file has to be changed. This is from our /usr/share/postfixadmin/languages/en.lang file: $PALANG['pUsersVacation_subject_text'] = 'Re: $SUBJECT'; ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 23:13:15
|
Patches item #3536134, was opened at 2012-06-18 16:13 Message generated for change (Tracker Item Submitted) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: making original subject referable in the vacation mail Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3536134&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 23:02:54
|
Bugs item #3536132, was opened at 2012-06-18 15:53 Message generated for change (Comment added) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&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: Vacation Group: v2.3.5 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: original subject should be referable vacation mail, patch Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- >Comment By: Christoph Lechleitner (ibcl) Date: 2012-06-18 16:02 Message: Some more information on our patch: This patch was made towards vacation.pl as of debian's postfixadmin 2.3.5-1 package for wheezy (the current testing release). It replaces any '$SUBJECT' (that is, a dollar sign and the word subject in upper case letters) in the vacation mail subject template with the subject of the original e-mail. For example: If the vacation mail subject is set to 'Re: $SUBJECT', the vacation mail gets the same subject as a common reply mail created by the recipient using an e-mail application. To use this as the default subject template, the langauge file has to be changed. This is from our /usr/share/postfixadmin/languages/en.lang file: $PALANG['pUsersVacation_subject_text'] = 'Re: $SUBJECT'; ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-06-18 22:53:07
|
Bugs item #3536132, was opened at 2012-06-18 15:53 Message generated for change (Tracker Item Submitted) made by ibcl You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&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: Vacation Group: v2.3.5 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christoph Lechleitner (ibcl) Assigned to: Nobody/Anonymous (nobody) Summary: original subject should be referable vacation mail, patch Initial Comment: The good-old vacation package by Eric P. Allman, Marco d'Itri, Stefan Muenkner supports referencing the subject of the original e-mail in the vacation e-mail by opening the .vacation.msg with something like Subject: Re: $SUBJECT We are in the process of moving about 100 mailboxes from a 9 year old "legacy" setup to a new setup using postfixadmin (and dovecot, roundcube, ...) and have developed a patch that adds a similar feature to postfixadmin's vacation.pl. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3536132&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-30 17:32:29
|
Patches item #3523124, was opened at 2012-05-02 15:18 Message generated for change (Comment added) made by norm_brandinger You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3523124&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Norm Brandinger (norm_brandinger) Assigned to: Nobody/Anonymous (nobody) Summary: Enhance XMLRPC support. Initial Comment: xmlrpc.php: Add support for reply_type, interval_time, activeFrom, activeUntil for the set_away() function. Update returned variable names in other function calls so they match dh->init() calls. functions.inc.php: Add authentication support for $_SESSION['username'] ---------------------------------------------------------------------- >Comment By: Norm Brandinger (norm_brandinger) Date: 2012-05-30 10:32 Message: The motivation for this patch, along with some other updates that I haven't submitted yet is to get the squirrelmail postfixadmin plugin operational. Note that at this time I do have full integration between postfixadmin and squirrelmail using the XMLRPC interface. I suppose that the extra parameters (or perhaps all of the parameters) for setAway() can have default values. That would make the call more backward compatible. For example: public function setAway($subject, $body, $reply_type="Auto_Reply", $interval_time=86400, $activeFrom="2000-01-01 00:00:00", $activeUntil="2500-12-31 23:59:59") { The reason for adding $_SESSION['username'] was to fix an authentication problem. On this debian system implementation, both postfixadmin AND squirrelmail are co-resident (which is what I think is the root of the problem). The first XMLRPC call appeared to succeed but subsequent calls failed with authorization errors. I'm running: PHP 5.3.3-7+squeeze9 with Suhosin-Patch (cli) (built: May 8 2012 10:41:34) with a mostly standard php.ini file. Regards, Norm ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2012-05-27 10:40 Message: From the xmlrpc.php patch: - public function setAway($subject, $body) { + public function setAway($subject, $body, $reply_type, $interval_time, $activeFrom, $activeUntil) { I'm not sure if changing the xmlrpc interface is a good idea - what happens if an "old" client connects and only provides subject and body? David, what's your opinion on this? (see also the recent discussion on the mailinglist - maybe we'll (have to) add another parameter for the realname) > functions.inc.php: > > Add authentication support for $_SESSION['username'] Norm, what's the reason for / advantage of this change? (it seems unrelated to the xmlrpc interface) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3523124&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 20:27:44
|
Bugs item #3094804, was opened at 2010-10-25 05:35 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3094804&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: None >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: passwords are escaped - ' and " let folder creation fail Initial Comment: create-mailbox: If a password contains ' or ", the IMAP login done by postfixadmin to create some folders fail because the password is escaped (\' and \"). We should hand over non-escaped password to create_mailbox_subfolders() and (most probably) pacrypt(). Important: We need to check if we need escaping before handing over the password to dovecotpw. Reported by gencha_ on IRC. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 13:27 Message: Fixed in SVN trunk r1397 + r1398. ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 12:18 Message: The escaping also breaks the resulting password hash, see https://sourceforge.net/tracker/?func=detail&aid=3301752&group_id=191583&atid=937964 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3094804&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 19:19:02
|
Bugs item #3301752, was opened at 2011-05-13 13:10 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3301752&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.3 >Status: Closed >Resolution: Duplicate Priority: 5 Private: No Submitted By: Paolo Stancato (seniormojo) Assigned to: Nobody/Anonymous (nobody) Summary: Problem with escape_string in pacrypt() Initial Comment: Hi there! Testing postfix admin I've found a bug with users authentication. The problem is that pacrypt() escape the password before return it and that can taint the hash. I've resolved the issue commenting the line functions.inc.php: 1244: $password = escape_string ($password); Data used for testing: --------------------------------- Password string: P4ssw0rd! Password hash: $1$>X6mz76\$EdT.4mI8ZEntI9/AgqazS1 ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 12:19 Message: Known bug, see https://sourceforge.net/tracker/?func=detail&aid=3094804&group_id=191583&atid=937964 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3301752&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 19:18:29
|
Bugs item #3094804, was opened at 2010-10-25 05:35 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3094804&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: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: passwords are escaped - ' and " let folder creation fail Initial Comment: create-mailbox: If a password contains ' or ", the IMAP login done by postfixadmin to create some folders fail because the password is escaped (\' and \"). We should hand over non-escaped password to create_mailbox_subfolders() and (most probably) pacrypt(). Important: We need to check if we need escaping before handing over the password to dovecotpw. Reported by gencha_ on IRC. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 12:18 Message: The escaping also breaks the resulting password hash, see https://sourceforge.net/tracker/?func=detail&aid=3301752&group_id=191583&atid=937964 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3094804&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 19:14:47
|
Bugs item #3034389, was opened at 2010-07-25 12:10 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3034389&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: Works For Me Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: dovecot 2.0 breaks dovecotpw encryption Initial Comment: copied from http://sourceforge.net/tracker/index.php?func=detail&aid=3032816&group_id=191583&atid=937964 ***** mickaelnguyen ***** 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~ ***** christian_boltz ***** 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.) ***** mickaelnguyen ***** 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: 2012-05-28 12:14 Message: I'm closing this as "works for me". To summarize the previous comments: If you use dovecot 2.x, you'll have to set $CONF['dovecotpw'] = '/path/to/doveadm pw' doveadm from dovecot 2.0.0 to 2.0.7 will not work because it checks if the input comes from a tty. You have to use at least dovecot 2.0.8. dovecot 1.x is still supported - in this case, just use $CONF['dovecotpw'] = "/usr/sbin/dovecotpw" > Have a look at > http://git.gauner.org/vboxadm.git/blob/HEAD:/contrib/roundcube/plugins/vboxadm/vboxadm.php, > the code there does exactly what you want to do. I had a short look - it might be a nice start, but it doesn't seem to support all encryption schemes available in dovecot - and in some cases, it also uses dovecotpw/doveadm pw... ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2011-11-25 12:50 Message: (Sourceforge trackers are somewhat special - IIRC they only allow the person who opened a tracker item to upload files. Like it or not...) Thanks for the script, but it shouldn't be needed. Instead, use $CONF['dovecotpw'] = '/path/to/doveadm pw' For the records: doveadm from dovecot 2.0.0 to 2.0.7 will not work because they check if the input comes from a tty. You have to use at least dovecot 2.0.8. ---------------------------------------------------------------------- Comment By: Lars Braeuer (mpexnetworks) Date: 2011-11-25 06:25 Message: I couldn't find a way to add an attachment other than opening a new issue, so here's the script: cat <<EOF >> /tmp/dovecotpw #!/bin/bash # MPeXnetworks - Lars Braeuer 11/2011 # Rebuild dovecotpw's original command line options, which are: #usage: dovecotpw [-l] [-p plaintext] [-s scheme] [-u user] [-V] # -l List known password schemes # -p plaintext New password # -s scheme Password scheme # -u user Username (if scheme uses it) # -V Internally verify the hash while getopts ":l:p:s:u:V:" opt; do case "$opt" in l) list=" -l" ;; p) plaintext=" -p $OPTARG" ;; s) scheme=" -s $OPTARG" ;; u) user=" -u $OPTARG" ;; V) verify=" -V" ;; esac done /usr/bin/doveadm pw ${list}${plaintext}${scheme}${user}${verify} exit $? EOF Move it to /usr/sbin/dovecotpw afterwards, if you are sure you are not overwriting an existing dovecotpw! ---------------------------------------------------------------------- Comment By: Lars Braeuer (mpexnetworks) Date: 2011-11-25 06:23 Message: I wrote a small bash wrapper script I placed in /usr/sbin/dovecotpw that calls doveadm pw. See the attached file. It's working fine for me without any change of postfixadmin for example. Haven't tested it with any other tools. ---------------------------------------------------------------------- Comment By: tex (tex0000) Date: 2011-03-18 13:11 Message: This is exactly the reason why I think that it's a _bad_ idea to rely on external binaries. If they are available at all they are going to break a some point in time. Why don't you implement the functionality in PHP? Have a look at http://git.gauner.org/vboxadm.git/blob/HEAD:/contrib/roundcube/plugins/vboxadm/vboxadm.php, the code there does exactly what you want to do. ---------------------------------------------------------------------- Comment By: Valkum (valkum) Date: 2010-12-30 10:28 Message: Timo said it was a bug in this version youve tested. A tty check is also done by getpass() so it is not the real problem. He said also that the tty check is gone in 2.0.8 For us it means that we need a way to suppress the output of /dev/tty which comes from dovecotpw and doveadm pw for use in cli,. ---------------------------------------------------------------------- Comment By: Valkum (valkum) Date: 2010-12-30 09:48 Message: Damn, checked it and it doesn't work ---------------------------------------------------------------------- Comment By: Valkum (valkum) Date: 2010-12-30 09:44 Message: Another try: Snowleopard wrote this at proc_open He uses this for his GPG function: // Set up the descriptors $Descriptors = array( 0 => array("pipe", "r"), 1 => array("pipe", "w"), 2 => array("pipe", "w"), 3 => array("pipe", "r") // This is the pipe we can feed the password into ); we could also try // Set up the descriptors $Descriptors = array( 0 => array("pty"), 1 => array("pty"), 2 => array("pty"), 3 => array("pty") // This is the pipe we can feed the password into ); But i think the securest and better way is to write a php lib for dovecot crypt methods ---------------------------------------------------------------------- Comment By: Charles (libertytrek) Date: 2010-12-30 07:17 Message: Maybe someone could pose a question to Timo (dovecot author) and he could make a suggestion for how to handle this - or possibly even change the way 'doveadm -pw' works so this could be fixed properly in pfadmin? ---------------------------------------------------------------------- Comment By: Valkum (valkum) Date: 2010-12-30 06:02 Message: I tested the pty fix but it doesn't work i got can't encrypt password with dovecotpw on my console with use of PFA-CLI ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2010-07-25 12:19 Message: A user comment on http://php.net/manual/en/function.proc-open.php (from 2007-12-27) suggests that using 'pty' might work. Please open functions.inc.php, search for "$spec = array(" (around line 1213 in 2.3.1) and replace it (including the following 3 lines) with: $spec = array( 0 => array('pty'), 1 => array('pty'), 2 => array('pty') ); Does this help? Otherwise we'll probably have to use "doveadm pw -p foobar" with the password on the commandline. However I don't like this idea because the password will be visible in the processlist for a short time then :-( ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3034389&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 19:10:11
|
Feature Requests item #2332595, was opened at 2008-11-23 04:15 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937967&aid=2332595&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: Interface Improvements (example) Group: SVN (please specify revision!) >Status: Closed >Resolution: Wont Fix Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: add "show password" link for mailboxes Initial Comment: SVN r482 introduced showing passwords in edit-mailbox.php - but in a quite broken way because it relied on an browser extension to change the stars in the password field to readable text. r482 was therefore reverted. We should add this feature in a way that works for everybody. (Needless to say that this is only useful for non-encrypted passwords ;-) Some sniplets and ideas from the mailinglist: add a "show password" link (with a $CONF setting to enable it, default should be disabled) which displays the password somewhere (using flash_info or a alert() feeded with a AJAX request) This would have some advantages: - it works in every browser and does not depend on special browser extensions - it only transfers the password on request - which reduces the risk (and number) of passwords in browser cache etc. a lot - it would easily allow to mail a notification to the user, which might be required by some people/companies for privacy or policy reasons > (So far, Postfixadmin is AJAX free, perhaps it should stay this way?) I don't want to do "big" things with AJAX. But I don't see a problem in using something like alert($password_fetched_by_AJAX_request>) The fallback could be that the "show password" link uses flash_info for this - with the disadvantage of causing a page reload (and possibly loose changes). ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 12:10 Message: Thinking about this again, I'm finally closing this request as "wontfix". Reasons are: - the password would be available in the browser cache - if someone can read your cache, then he'll be able to steal the password. - it would only work with cleartext passwords - the idea of an additional clear_password column makes things even worse - if the config says to encrypt the password, then admins will kill us for having an additional clear_password column ---------------------------------------------------------------------- Comment By: Bjorn (bjornnilsen) Date: 2009-05-21 19:58 Message: I have a requirement for this feature and I was able to implement is very simply. I have tried to keep the changes to an absolute minimum so that I can easily add it into new releases as they come out. I was very impressed at how simple it was to add this feature. I guess this shows postfixadmin has been designed very well :) ALTER TABLE `mailbox` ADD `clear_password` VARCHAR( 255 ) NOT NULL ; vi edit-mailbox.php FIND: $tName = $user_details['name']; ADD: $tClearPassword = $user_details['clear_password']; FIND: $formvars['password'] = pacrypt($fPassword); ADD: $formvars['clear_password'] = $fPassword; vi templates/edit-mailbox.php FIND: <tr> <td><?php print $PALANG['pEdit_mailbox_username']; ?></td> <td><?php print $fUsername; ?></td> <td><?php print $pEdit_mailbox_username_text; ?></td> </tr> ADD: <tr> <td>Password</td> <td><?php print $tClearPassword; ?></td> <td></td> </tr> ---------------------------------------------------------------------- Comment By: Pavel Volkovitskiy (olfway) Date: 2009-03-03 01:35 Message: why would you open edit-mailbox if you don't want to change name/password? so user should be notified if someone open it mailbox in edit mode if you care about security you should use https to avoid passwords sniffing also i'm not sure that you can copy password from js alert box the real issue was only missing check if password stored in clear text or not also, you can't always change password for user b/c then you will have to change settings on every users pc/gadget ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937967&aid=2332595&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 18:51:52
|
Feature Requests item #3006020, was opened at 2010-05-23 07:53 Message generated for change (Settings changed) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937967&aid=3006020&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: None >Status: Pending >Resolution: Fixed Priority: 5 Private: No Submitted By: Valkum (valkum) Assigned to: Nobody/Anonymous (nobody) Summary: Advanced Config and Lang Initial Comment: I think about use an extended version of Config and Lang variables. In my CLI I use a simple class, cloned from Joomla!. I think this should be implemented in functions.inc.php. With this class you can read even lang and conf option in classes. This is not working with globals. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 11:51 Message: IMHO the implementation is (mostly) done and will progress while moving everything to the *Handler classes. If I'm really bored one day ;-) I'll also check functions.inc.php and change everything to use the Config and Lang classes there. However, I don't see the need to keep this request open. If you disagree, please speak up - otherwise this request will be auto-closed in 4 weeks. ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2011-10-27 13:01 Message: Well... ;-) gingerdog wrote: > Changing the configuration to be in an object which is a singleton and > silently used everywhere isn't really any different to using a global array > - it's still "a magic thing" which the callee/user has no direct control > over. The code has a high coupling to Config (the class) just in the same > way as it had a high coupling to the $CONF array. We won't get dependency-free code, and I don't see a problem with depending on the Config class. (And I don't think it's a problem that the caller/user has no direct control over it.) As a general note: My reason for migrating functionality to the *Handler classes is not to make the code object-orientated, MVC-compliant, encapsulated etc. The real reason for me is to have _easy to maintain_ code. And that's something where classes are great - we can move the code that is common for all *Handler classes to PFAHandler and it's available in all *Handler classes automatically. And we still have the flexibility to override some parts by (re)defining them in each *Handler class. The AliasHandler will probably only need the functions initStruct and initMsg, MailboxHandler will need some more (send welcome mail etc.). If we get object-orientated, MVC-compliant, encapsulated etc. code as a side effect: Nice, why not. If I have to ignore some holy grail rule of object-orientated programming to reach the target "easy to maintain code", then I'll just ignore the rule ;-) I can already see you screaming when I implement hooks for editing/customizing the $struct array ;-)) but nevertheless those hooks might be one of the most important improvements in 3.0 because they make it easy to add custom fields, hide existing fields etc. > Really the model classes should support injection of the Config/Lang > objects so they can be (if necessary) swapped for something else to aid > testing and reuse. You can easily replace the content of Config/Lang using Config::write/Lang::write, so I don't see a real problem here. > If Lang has no functionality over/above a Config object, is it worth > having having - why not just store the stuff in Config? One comment earlier you complained about "not really clean" code ;-) From the technical side we don't need a separate object for the texts and could store them in Config - but texts are not config options. I'd say we keep separate objects if they don't cause any problems (I have a server with PHP 5.2.14 and will test the behaviour there when I find some time.) valkum wrote: > What about changing the whole Lang object to use a helper function > t("string") That doesn't change much (except saving some bytes), but makes the code harder to understand. I prefer human-readable function/class calls. _If_ we decide to use the Config object for the texts, using something like Config::Lang('string') might be the best solution. ---------------------------------------------------------------------- Comment By: Valkum (valkum) Date: 2011-10-26 16:55 Message: What about changing the whole Lang object to use a helper function t("string") t("string") looks in Conf for language and include_once the matching language file when it is not already included. then t() returns the matching string ---------------------------------------------------------------------- Comment By: GingerDog (gingerdog) Date: 2011-10-25 16:14 Message: (I'm also not sure how the Lang class will actually behave, especially under PHP5.2 where there isn't late static binding. I have a feeling calling Lang::getInstance() may actually return a Config object. If Lang has no functionality over/above a Config object, is it worth having having - why not just store the stuff in Config? ---------------------------------------------------------------------- Comment By: GingerDog (gingerdog) Date: 2011-10-25 16:12 Message: <2p> To be argumentative - Changing the configuration to be in an object which is a singleton and silently used everywhere isn't really any different to using a global array - it's still "a magic thing" which the callee/user has no direct control over. The code has a high coupling to Config (the class) just in the same way as it had a high coupling to the $CONF array. Really the model classes should support injection of the Config/Lang objects so they can be (if necessary) swapped for something else to aid testing and reuse. </2p> ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2011-10-25 16:02 Message: In the meantime I moved your classes to model/ and removed duplicate code - the "Lang" class shrunk to class Lang extends Config { # exactly the same code, just another name ;-) } ;-) Some functions already use Lang::read and Config::read, the others will follow sooner or later. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937967&aid=3006020&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 18:32:12
|
Bugs item #3519646, was opened at 2012-04-19 14:25 Message generated for change (Settings changed) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3519646&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Juraj Lutter (wilbury) Assigned to: Nobody/Anonymous (nobody) >Summary: vacation: subject not being encoded Initial Comment: Please use encode_mimewords() for $subject in %mail. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3519646&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 18:24:28
|
Bugs item #2646447, was opened at 2009-02-27 12:03 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2646447&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: None Group: None >Status: Closed >Resolution: Rejected Priority: 5 Private: No Submitted By: Pavel Volkovitskiy (olfway) Assigned to: Nobody/Anonymous (nobody) Summary: show admin's password too Initial Comment: if $CONF['show_password'] == "YES" then show admin's password too PS: there is extension for FF "Unhide Passwords" so passwords will be in "****" form until you clicked on password field ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 11:24 Message: This is not what $CONF[show_password] was made for ;-) From config.inc.php: // Show Password // Always show password after adding a mailbox or admin. // If you want to always see what password was set set this to 'YES'. $CONF['show_password'] = 'NO'; Your patch would include the password in the edit-admin form, visually obscured as '*******' - but it would be available in clear text if someone uses "view source" or (bigger risk) reads your browser cache. In other words: It would make PostfixAdmin insecure! Therefore I'm rejecting this patch. If you really want this feature (IMHO it's a mis-feature), you'll have to apply to the patch to your installation of PostfixAdmin yourself. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2646447&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 18:14:36
|
Bugs item #2886179, was opened at 2009-10-25 16:13 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2886179&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: Database Group: v 2.3 >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: domain changes not logged Initial Comment: enabling/disabling a domain, creating and editing a domain are not logged to the log table. Found by p5yk0tik, see https://sourceforge.net/projects/postfixadmin/forums/forum/676076/topic/3406252 ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 11:14 Message: This was accidently ;-) fixed in SVN trunk by using DomainHandler instead of the old create-domain.php/edit-domain.php scripts some months ago. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=2886179&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 18:10:39
|
Bugs item #3039042, was opened at 2010-08-03 14:55 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3039042&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: None Group: v2.3.1 >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: gw0 (gw0) Assigned to: Nobody/Anonymous (nobody) Summary: absolute vs. relative "Location:" redirects Initial Comment: I noticed that every time I try to add a new domain alias, the following page opens "/postfixadmin/create-alias-domain.php" to display it successfully done its job. Unfortunately I am using a reverse proxy on some other URL (not /postfixadmin/) that opens pages on the server where Postfix Admin lives (here it is in /postfixadmin). All other pages and functions seem to work fine, problems are just with adding new domain aliases. It seems that there is a hardcoded URL of some sort and it would be better to use relative links like everywhere else. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 11:10 Message: Finally fixed in SVN trunk (commits: r1394, r1395, r1396) I removed the config option $CONF['postfix_admin_url'] - it is no longer needed. ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2011-04-09 15:15 Message: I changed most of the Location: headers to relative paths in SVN trunk r1021. "most" means that some not-so-easy cases still need to be fixed. $CONF['postfix_admin_url'] is still used in: - functions.inc.php - authentication_require_role() - smarty.inc.php - as prefix for $CONF['theme_css'] and $CONF['theme_logo'] - and obviously config.*.php ---------------------------------------------------------------------- Comment By: GingerDog (gingerdog) Date: 2010-09-14 01:31 Message: Use relative URLs where possible - ideally we could get rid of $CONF['postfix_admin_url'] then. ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2010-09-11 16:48 Message: Just a technical explanation: create-alias-domain.php has (technically correct): header("Location: " . $CONF['postfix_admin_url'] . "/main.php"); edit-alias.php has (not really correct because redirects have to use absolute URLs, but it seems to work with all browsers even with relative paths) header ("Location: list-virtual.php?domain=$fDomain"); oh, and create-alias does not do any redirects ;-) That said: we should use absolute _or_ relative redirects, but not a mix of it. Funnily, the technically wrong solution (relative redirects) tend to be the better solution because they work even if $CONF['postfix_admin_url'] is incorrect. It seems all browsers accept it since nobody complained yet that for example the edit-alias redirect doesn't work. GingerDog: what is your opinion about this? ---------------------------------------------------------------------- Comment By: gw0 (gw0) Date: 2010-08-03 15:07 Message: Ok, it seems $CONF['postfix_admin_url'] is being used and I misconfigured it. It is weird though that adding normal aliases works fine... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3039042&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-28 17:29:08
|
Bugs item #3292648, was opened at 2011-04-25 07:19 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3292648&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: None Group: SVN (please specify revision!) >Status: Closed >Resolution: Fixed Priority: 9 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: list-virtual: pagebrowser broken for postgreSQL Initial Comment: To avoid this gets lost/forgotten: The pagebrowser in list-virtual is broken for postgreSQL since SVN trunk r1033. This means list-virtual breaks if more than $CONF[page_size] mailboxes/aliases exist for a domain. David, please fix the SQL queries in create_page_browser() in functions.inc.php. (If someone else can provide working queries, I'll also accept them of course ;-) ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-28 10:29 Message: I finally installed pgsql and fixed the queries (commited to SVN r1392). To my surprise, the problem was (only) caused by the count() query. The query that generates the pagebrowser already worked. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3292648&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-27 19:09:45
|
Bugs item #3066059, was opened at 2010-09-14 09:32 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3066059&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: None Group: v2.3.2 >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Christian Boltz (christian_boltz) Assigned to: Nobody/Anonymous (nobody) Summary: space in aliases shouldn't be allowed Initial Comment: Accidential ;-) finding by a customer: When creating an alias, you can enter " foo" in the from field. This results in the creation of an " fo...@ex..." alias (with leading space!) and people will wonder why the alias will fail with "user unknown". PostfixAdmin should prevent spaces - either with trim() or with a error message. ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-27 12:09 Message: Fixed in SVN trunk r1390 - check_email() did a trim() before validating the mail address :-/ @robelix: I can't create a domain with space at the end in SVN trunk, therefore I assume it's fixed. ---------------------------------------------------------------------- Comment By: roland obermayer (robelix) Date: 2011-06-16 09:21 Message: Just had a pretty similar issue: A domain "example.com " - with a space at the end - was added. It seems that postfix does not mind - mails are. But in postfixadmin it's impossilbe to select it in the virtual-list. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3066059&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-27 17:40:02
|
Patches item #3523124, was opened at 2012-05-02 15:18 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3523124&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: Vacation Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Norm Brandinger (norm_brandinger) Assigned to: Nobody/Anonymous (nobody) Summary: Enhance XMLRPC support. Initial Comment: xmlrpc.php: Add support for reply_type, interval_time, activeFrom, activeUntil for the set_away() function. Update returned variable names in other function calls so they match dh->init() calls. functions.inc.php: Add authentication support for $_SESSION['username'] ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-27 10:40 Message: From the xmlrpc.php patch: - public function setAway($subject, $body) { + public function setAway($subject, $body, $reply_type, $interval_time, $activeFrom, $activeUntil) { I'm not sure if changing the xmlrpc interface is a good idea - what happens if an "old" client connects and only provides subject and body? David, what's your opinion on this? (see also the recent discussion on the mailinglist - maybe we'll (have to) add another parameter for the realname) > functions.inc.php: > > Add authentication support for $_SESSION['username'] Norm, what's the reason for / advantage of this change? (it seems unrelated to the xmlrpc interface) ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3523124&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-27 17:31:32
|
Bugs item #3436282, was opened at 2011-11-10 16:57 Message generated for change (Comment added) made by christian_boltz You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3436282&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: Interface (example) Group: SVN (please specify revision!) >Status: Pending Resolution: None Priority: 5 Private: No Submitted By: Jonathan Dray (spiroid) Assigned to: Nobody/Anonymous (nobody) Summary: No pagination links for alias list Initial Comment: When more than 10 aliases are stored in the database, only the 10 first are displayed. There are no links to navigate and to go to the next 10 elements. Postfixadmin svn installed, rev : r1263 ---------------------------------------------------------------------- >Comment By: Christian Boltz (christian_boltz) Date: 2012-05-27 10:31 Message: Can you please test again with latest SVN? I fixed an issue with "mysqli" in r1388 on 2012-05-13 which most probably caused this bug. BTW: "mysql" without the "i" was not affected, which explains why I couldn't reproduce this. ---------------------------------------------------------------------- Comment By: d-fens_ () Date: 2012-05-11 04:52 Message: hi, i have the same problem with latest svn and mysql, no prev/next links in aliases and templates ---------------------------------------------------------------------- Comment By: Christian Boltz (christian_boltz) Date: 2012-04-09 13:42 Message: I can't reproduce that with current SVN r1366 and MySQL as database. OTOH, the pagebrowser is broken in postgresql (known issue, https://sourceforge.net/tracker/?func=detail&aid=3292648&group_id=191583&atid=937964 ) Can you still reproduce this bug? If yes, which database are you using? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937964&aid=3436282&group_id=191583 |
From: SourceForge.net <no...@so...> - 2012-05-24 23:35:30
|
Patches item #3529620, was opened at 2012-05-24 16:35 Message generated for change (Tracker Item Submitted) made by jan-kruis You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3529620&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: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: J.Kruis (jan-kruis) Assigned to: Nobody/Anonymous (nobody) Summary: real name [update] Initial Comment: Hi David I have updated my realname patch get lose of my rubbish also i have test it and works fine on the mail versie 4.5 (mac OS 10), iphone ,microsoft Entourage en Outlook (mac) i think on windows al well. And as you can see i dont use fake_from I run into a problem i can't get the check working on an empty real_name to exchane it then with the email adress any help i also put in some extra's -error checking on setting an Active unit is before Today which now gives a errormsg and stays unit is is corrected. -userscontrol for domain admins - text adjustment in nl.lang - typo error in vacation.pl on line 407. [vacaton schoot be vacation which cause a error if email is alais] - DomainHander.php makes a difference in the flashmsg between create and edit a domain. BTW Table vaction required a record real-name same struc. as record email. question Shoot there be a $CONF[realname] = yes in config.inc.php ? please have a look and let me now regards Jan ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=937966&aid=3529620&group_id=191583 |