25 Jan
2010
25 Jan
'10
4:44 p.m.
On Monday 25 January 2010, Ovidiu Sas wrote:
I'm in favor of keeping this module as a db only
module. Combining
both init methods (param and db config) is messy due to the current
module initialization (the shm mem is not available during the param
parsing). For small/embedded systems db_text is always a viable alternative.
Hello Ovidiu,
i did not reviewed the code yet, but some people also like to run their
balancers without a database to avoid this additional complexity.
In the future, the ratelimit module should become
obsolete and only
pipelimit should be maintained.
Sure, if the functionality that most people use in ratelimit is also available
there, it makes no sense to keep both.
Also, I'm in favor of keeping the new module as
simple as possible and
perform any fancy operations in the config file (like regex to pv via
textops).
I agree, this should be all done in the configuration, its much easier to
maintain.
Cheers,
Henning