Skip to content

"Explicit" rules#13

Open
hanshuebner wants to merge 12 commits intonand2:masterfrom
goeuro:master
Open

"Explicit" rules#13
hanshuebner wants to merge 12 commits intonand2:masterfrom
goeuro:master

Conversation

@hanshuebner
Copy link

Hi,

this pull request implements some changes to implement explicit rules. Before, rules were always created implicitly by the is_allowed call. This is suitable for the rate limiting use case that motivated the module in the first case, i.e. limiting specific clients. I need the module to put a (dynamic) limit on the backend, though, in order to prevent overload situations.

With these changes, it is possible to change the windows definition for a rule after it has been initially created (that would not work before). Also, a new call set_windows has been added that can be used to create a rule or re-set the limits for a certain rule.

As changes to such explicit rules are an administrative measure, I have added code to log rule changes for explicit rules. Also, the module now logs errors arising from invalid window definition strings using syslog.

Thanks for the module, it works great!
Hans

@nand2
Copy link
Owner

nand2 commented May 22, 2014

Hello!

Sorry it's been so long on my mailbox!
Unfortunately I'm having a unexpected project which is eating all my free time :-/
Your modifications seem quite interesting and substantial, and I would need to review and test everything with those, but I cannot right now :-/
I'm seeing forks of libvmod-throttle, for Varnish 4.0, and with more bugfixes, which seems to be taking the lead, maybe it would be good to merge on these forks.

Sorry about that!
Nand

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants