Dev

dev@lists.roundcube.net

1 participants
4289 discussions

php code design/performance
by darklight 06 Jul '09

06 Jul '09

Hi, walking through roundcube's code I've found some room for small improvements. if you sum them all together you may get some nice performance improvement (not very huge - but you can probably notice it) what I'm talking of are mainly style questions... because there are functions in php that do the same as another function, with the small difference that function X is faster for example print() is slower than echo() also double-quotes (") are up to 25% slower than concatenating strings with single quotes (') - which is a HUGE performance boost I've found a site which lists some of those functions - check [1] I quickly compared the list with roundcube's codebase those are the main places where there is still room for improvement: * use ' instead of " (where possible - for example 'this is a test' . "\n" would still be faster than "this is a test\n" - but you cannot replace '"\n" by '\n') * multiple parameters for echo - but most people prefer concatenating over this multiple argument feature (I have to admit: I also prefer string concatenation) * pre-increment instead of post-increment (should be quite easy to do this, since you need post-increment only in very few cases) * unset() should be used more often (bigger task) * use require() instead of require_once() - this is probably a bigger task too one small note about [1]: afaik switch/case is faster than if/else that site says the opposite some of those things can be put into roundcube's codebase quite easily maybe you can create some "Junior Jobs" section in your wiki, so bored devs may write patches (which of course should be reviewed by an experienced dev) however, feel free to discuss this oh, and I know: this whole mail is not mean for today's development... but more something for the future Regards, Martin [1] http://progtuts.info/55/php-optimization-tips/ _______________________________________________ List info: http://lists.roundcube.net/dev/

5 6

managing translations
by Martin 06 Jul '09

06 Jul '09

Hi, I always read about localization updates on your mailing list I started asking myself how translators know that they have to translate something which was recently added? do they check the interface and see "oh, there's text which is not translated yet"? or do they compare their locale with en_US ? anyway, wouldn't it be easier if some (maybe I :)) wrote some small PHP script that could easily compare multiple localized include-files with each other? if you want something like this feel free to tell me Regards, Martin _______________________________________________ List info: http://lists.roundcube.net/dev/

2 1

Re: [RCD] Message archive plugin
by Roland Liebl 06 Jul '09

06 Jul '09

Thanks Thomas! The hook has meanwhile changed to "render_mailboxlist". It works great on the mailbox list within the task "mail". It does not work on the subscription table. Regards, Roland _______________________________________________ List info: http://lists.roundcube.net/dev/

1 0

High cpu overload
by Tiago Gomes 05 Jul '09

05 Jul '09

Hello everyone. sorry because my poor english congratulations to all the excellent project roundcube mail. I have one problem, I installed all versions for test, including svn / trunk (latest) and all creates overload on my server. High cpu load I'm using the services: Apache: Server version: Apache/2.2.6 (Unix) Server built: Jul 1 2009 20:41:05 * - -with-mpm=worker* Php: PHP 5.2.6-1+lenny3 with Suhosin-Patch 0.9.6.2 (cli) (built: Apr 26 2009 22:16:23) Copyright (c) 1997-2008 The PHP Group Zend Engine v2.2.0, Copyright (c) 1998-2008 Zend Technologies Mysql 5.0 SERVER load average info after 10 minutes then restart the apache server: *top - 16:48:25 up 6:24, 3 users, load average: 6.92, 2.89, 3.88* Tasks: 84 total, 2 running, 82 sleeping, 0 stopped, 0 zombie *Cpu(s): 98.1%us, 1.7%sy, 0.0%ni, 46.9%id, 0.2%wa, 0.0%hi, 0.0%si, 0.0%st* Mem: 2076288k total, 1079936k used, 996352k free, 297020k buffers Swap: 498004k total, 0k used, 498004k free, 486532k cached PID USER PR NI VIRT RES SHR S *%CPU* %MEM TIME+ COMMAND 14103 www-data 20 0 564m 125m 3536 S *190* 6.2 6:28.57 httpd 14088 www-data 20 0 517m 89m 3464 S * 118* 4.4 10:45.10 httpd the log error log from round roundcube mail: Only: *[03-Jul-2009 16:48:08 -0300] DB Error: MDB2 Error: constraint violation Query: _doQuery: [Error message: Could not execute statement] [Last executed query: EXECUTE mdb2_statement_mysql_501fd0fc317c477d1c729158644dc2c5bd5f1d12d USING @0, @1, @2, @3, @4, @5, @6, @7, @8, @9, @10] [Native code: 1062] [Native message: Duplicate entry '2557-INBOX.msg-13334' for key 2] in /webhost/ webmail.atarde.com.br/program/include/rcube_mdb2.php on line 621 (GET /?_task=mail&_mbox=INBOX&_page=1) [03-Jul-2009 16:48:08] MDB2 Error: constraint violation (-3): _doQuery: [Error message: Could not execute statement] [Last executed query: EXECUTE mdb2_statement_mysql_674a5a0876c6968d9d29cb5837c47123866f39a43 USING @0, @1, @2, @3, @4, @5, @6, @7, @8, @9, @10] [Native code: 1062] [Native message: Duplicate entry '2557-INBOX.msg-13401' for key 2] * Can you Help me, ? Thank's -- ---------------------------------------------------------------- Tiago Gomes Pereira http://www.tiagogomes.eti.br LPI Certified LPI ID - LPI000129445 IT Service Management - ITIL FOUNDATION V2 ID - SR341901 algodas(a)gmail.com _______________________________________________ List info: http://lists.roundcube.net/dev/

2 2

updated pt_BR localization
by Denis Volpato Martins 05 Jul '09

05 Jul '09

Hi, I’ve attached the patch for the completed pt_BR localization. Att. - Denis --- 8< --- detachments --- 8< --- The following attachments have been detached and are available for viewing. http://detached.gigo.com/rc/Bf/a6SuCK11/pt_BR.patch Only click these links if you trust the sender, as well as this message. --- 8< --- detachments --- 8< --- _______________________________________________ List info: http://lists.roundcube.net/dev/

1 0

Re: [RCD] High cpu overload
by till 04 Jul '09

04 Jul '09

Oops... Btw, please CC the list (always). On Sat, Jul 4, 2009 at 10:50 PM, till<klimpong(a)gmail.com> wrote: > On Sat, Jul 4, 2009 at 10:09 PM, Tiago Gomes<algodas(a)gmail.com> wrote: >> yes, I do , but the cpu HIGH Over continued, any Idea ? it ´s normal ? > > I hope that means the database error is gone. ;-) > > In general, RoundCube doesn't cause high loads for me. I'd suggest you > run xdebug and generate some cachegrind files (through xdebug's > profiler) to figure out what is going on. There could be a number of > things slowing the webserver down. > > http://xdebug.org/docs/profiler > > Till > _______________________________________________ List info: http://lists.roundcube.net/dev/

1 0

Re: [RCD] Message archive plugin
by Roland Liebl 03 Jul '09

03 Jul '09

Is there a way to localize the "Archive" folder? Regards, Roland _______________________________________________ List info: http://lists.roundcube.net/dev/

3 2

Database Table for Plugins
by Roland Liebl 03 Jul '09

03 Jul '09

Hi all, I think it might be a good idea to have a database table for plugins. If plugins needs to store some values currently this is done mostly by using 'users' table 'preferences' field. I think it's better to have a separate table. The following structure should be flexible enough. ------------------------------------------------------ CREATE TABLE IF NOT EXISTS `plugins` ( `id` int(11) NOT NULL AUTO_INCREMENT, `user_id` int(11) DEFAULT NULL, `contact_id` int(11) DEFAULT NULL, `identity_id` int(11) DEFAULT NULL, `plugin` text, `key` text, `value` text, `timestamp` timestamp NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP, PRIMARY KEY (`id`) ) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=1 ; -------------------------------------------------------- Any comments are welcome. Regards, Roland _______________________________________________ List info: http://lists.roundcube.net/dev/

1 0

memory use
by Cor Bosman 29 Jun '09

29 Jun '09

Hi devs, just wondering if this is normal: Jun 29 16:36:03 roundcube-test roundcube: PHP Fatal error: Allowed memory size of 104857600 bytes exhausted (tried to allocate 17100392 bytes) in /data/WWW/roundcubemail-2493/program/lib/Net/SMTP.php on line 831 This is a 12MB attachment. That shouldnt be needing a total memory footprint of 100+ MB. I had to move our memory limit up to 200MB just to be able to send a 12MB attachment. Regards, Cor _______________________________________________ List info: http://lists.roundcube.net/dev/

1 0

One time ticket auth and imapproxy
by Robert King 26 Jun '09

26 Jun '09

Folks, I'm experiencing a very odd issue with a custom plug-in using the authenticate hook. Here at our university we use a Web based single sign on authentication called CAS (http://www.jasig.org/cas). I am trying to write a plug-in to allow CAS to work with Roundcube proxying a connection through imapproxy (http://www.imapproxy.org/). The normal operation for authentication for IMAP by CAS service is to get a one time use CAS ticket for every authentication. Since RC stores and reuses the initial password CAS cannot work in direction connection to the IMAP server. This is not an issue because when we proxy through imapproxy, it holds open the imap connection by password/username combination. So everything should be working great and superficially it seems okay, that is until you try and delete an e-mail. It seems that a delete will call the authenticate hook again and grab a new CAS ticket. The new CAS ticket breaks the imapproxy connection identifier because the password is now different. In turn this causes rc to have a "lost imap connection" error. The way I look at it I have 2 possibilities: 1. Figure a way out to only request the CAS ticket once in the plugin. I'm guessing do a check for existing ticket before requesting. This is the quick fix, but feels like a "duct tape" solution for what might be a larger issue in extending RC for alternative authentication mechanisms. 2. Extend RC so that instead of caching a password, it uses a password recall function that can has an exec handler for custom plug-ins $rcmail->auth->getpass() returns a clear text password and has an exec handler for password return. Default it would be an abstraction of the decrypt session password to keep current functionality The value with option two is it would allow plug-in authors to authenticate to peripheral services like managesieve with out limiting alternative authentication methods. I've included the current sort of working plugin for anyones interest. -- Robert King System Administrator Computing & Communications Memorial University --- 8< --- detachments --- 8< --- The following attachments have been detached and are available for viewing. http://detached.gigo.com/rc/BK/UGmCcQQp/casauth.php Only click these links if you trust the sender, as well as this message. --- 8< --- detachments --- 8< --- _______________________________________________ List info: http://lists.roundcube.net/dev/

1 2

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

Dev