Hello,
I was testing the Flood Unblock module (https://www.drupal.org/project/flood_unblock) with a Redis Flood backend.
And the module is useless in this case because it makes direct queries against the database to visualize the flood.
I opened an issue on the module #2928007: Support external Flood (Redis, etc.) but to address it properly a change in Core needs to be done.
Would it be possible to add some methods to the FloodInterface (and so the database backend native service) to be able to request the flood backend generically so flood visualization would be possible?
If there is an interest in this feature and a GO from Core people, that there is hope that this feature request will land in core, I will try to make a patch for that.
Thanks for any reply.
Comments
Comment #2
dawehnerI think the better approach would be to have an additional interface, maybe extending the existing one. Then you could still implement a simple flood system, but then if you want, provide a queryable one.
Comment #3
grimreaperThanks @dawehner for the reply.
I am going to post two patches in the Issue on Flood unblock (one for Flodd unblock, the other one for Redis).
And I opened an issue on Redis #2928106: Flood: Compatibility with Flood unblock to request Redis maintainer help.
Should I close this issue and let the work be done in the issue on Flood control?
Comment #5
andypost++ to add
FloodIntrospectionInterfaceotoh maybe better to make it separate service?Comment #6
andypostComment #18
smustgrave commentedThank you for sharing your idea for improving Drupal.
We are working to decide if this proposal meets the Criteria for evaluating proposed changes. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or there is no community support. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!