- Issue created by @generalredneck
- π·πΈSerbia vaish
This would be great improvement. Are you planning on creating a MR? Note that issue β¨ [Patch provided] Allow administrators to specify IP addresses and ranges which should never be rate limited Needs review provides a setting for ignoring IP addresses. We should make sure that the structure of the settings array and naming of the array keys remains consistent and intuitive after both settings have been added. E.g. both setting should use the same verb, either "ignore" or "allow", etc.
- πΊπΈUnited States generalredneck
Not immediately. I may at some point in the future but I'm unable to secure a client to develop this against right away.
I was mostly poling around the code to see how stuff worked when I saw the issue...
At the very least we should have a hook here... that one configuration where you derive the path for styles is a use case for having it as a hook, but all the other paths are simple hard coded instances which lends themselves to configuration like you mentioned.
- π·πΈSerbia vaish
I would rather try to avoid invoking any hooks in this module. In order for the module to be really useful, it should be able to process each request as early and as fast as possible. Invoking hook adds an overhead which is avoidable. For the same reason I decided not to use Drupal config and create regular configuration page.
Thanks for the feedback and let me know if you happen to decide to work on this.
- πΊπΈUnited States generalredneck
Those are all good feedback. I believe it might be worth adding to a contribution section of the Readme.
Thanks for framing the ideas... it's a lot like the way fast404 works