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