Actually, as I was falling asleep last night (isn't that always the
case), I wondered why chkuser.c sets maxmbxquota_limit = 0 and not 100
by default.   It would seem to me if you're enabling the define, you
would already expect that function to just work, not go to another place
and enable something else. Having the environment variable is great, then if you want to alter the default, you can set it there...


Just my .02.

Rick

Rick

I believe that Tonino has set this for the tcp.smtp otherwise you need
to recompile qmail everytime you need to change the setting for the quota.

Just my 2 cents.

Remo
Right - that makes sense, but as it is now when it's enabled, it's not REALLY enabled until the environment is set. The environment is required, it's not an option. This is because in the chkuser.c the limit is set to 0, which disables the check. If, by default, the limit was set to 100, then it would be enabled by the define AND you can change the limit in environment or disable it by setting the environment to 0. To me the environment variables should override the 'standard' - and if you've enabled 'smtp bouncing', you shouldn't have to add the environment as well (imho, enabling it twice).

Rick



!DSPAM:475ebd8132003342933973!

Reply via email to