Re: [Postfixadmin-devel] Changing some defaults in trunk/3.0?

[Robert S. <ro...@sc...>]

Am 13.04.2012 02:04, schrieb Christian Boltz:
> Hello,
> 
> 3.0 will be a major release, which means I won't be too strict about 
> keeping the default config 100% backwards compatible (= unchanged).
> We probably have some config options with defaults that are there for 
> historic reasons, but are not the best idea nowadays. 
> 
> Are you aware of any setting in config.inc.php that is
> - outdated (as in "everybody has to change this nowadays")  or
> - "not a good idea"[tm]  or
> - could have better default values
> ?
> 
> If yes, please speak up now and tell me
> - which $CONF option?
> - why would you change its default value?
> - and to which value? (you can obviously ignore this part for YES/NO 
>   options ;-)
>  
> 
> Let me start with some things I noticed while scrolling over 
> config.inc.php:
> 
>     $CONF['database_type'] = 'mysqli'; # current value: "mysql"
> should be changed to "mysqli". The reason is simple - I don't expect we 
> have too many users using MySQL < 4.1 nowadays ;-) so using the (newer) 
> mysqli interface by default would make sense.

ACK

> 
>     $CONF['dovecotpw'] = "/usr/sbin/doveadm pw"; 
>     # current value: "/usr/sbin/dovecotpw";
> reason: dovecot 2.x comes with doveadm instead of dovecotpw

ACK

> 
>     $CONF['domain_path'] = 'YES'; # was NO
>     $CONF['domain_in_mailbox'] = 'NO'; # was YES
> in other words: domain/username/ instead of username@domain/
> reason: maybe personal taste ;-)

there might not be a default for all people satisfy
reading doku stays mandatory

> 
>     $CONF['quota'] = 'YES'; # was NO
> reason: I'm using quota everywhere - even if I make the mailbox *very* 
> big, just to avoid that a forgotten mailbox can fill up the harddisk.

NOTACK, using quota isnt so wide spreaded as you think, so stay simple
default might be better choice

> 
>     $CONF['vacation'] = 'YES'; # was NO
> why should we hide this feature? ;-)

NOTACK,
vacation is done via sieve ?

> 
>     $CONF['alias_control'] = 'YES'; # was NO
>     $CONF['alias_control_admin'] = 'YES'; # was NO
> I stopped counting how often users asked to implement this feature ;-) 
> so we should enable it by default

OK

> 
>     $CONF['backup'] = 'NO'; # was YES
> The backup module is basically unmaintained...

then it should stay NO

> 
>     $CONF['show_status']='YES'; # was NO
> Why should we hide such a nice feature?
> Note that some of the related options would also need better defaults, 
> but let's first discuss if enabling show_status by default makes sense.

just forgot what show status means *g

> 
>     $CONF['new_quota_table'] = 'YES'; # was NO
> new quota table format in dovecot 2.x
> 

ACK but depend on quota enable defaults

> 
> Needless to say that this *must* be documented in a very visible way to 
> avoid "surprising" users on upgrades ;-)
> 
> 
> Regards,
> 
> Christian Boltz

just my thoughts , as you asked, anyway thx for coding !

is there a changelog preview by new feature sets ?

-- 
Best Regards

MfG Robert Schetterer

Germany/Munich/Bavaria