Add Redis support

Created on 3 April 2024, 3 months ago
Updated 3 June 2024, 23 days ago

As described in ✨ Support external Flood Postponed: needs info the FloodUnblockManager contains database specific code which makes it unusable for Redis.

The patch in ✨ Support external Flood Postponed: needs info is problematic, so I propose a 2 step solution:

- 1. Add the generic FloodUnblockManager. ( ✨ Make FloodUnblockManager generic Active ) This will be achieved by adding the generic FloodUnblockManagerBase and the database specific FloodUnblockManagerDatabase. This should be done while ensuring that everything keeps functioning as it does in the latest release.
- 2. Add the Redis FloodUnblockManager. (This issue) This will be achieve by adding the FloodUnblockManagerRedis

✨ Feature request
Status

Active

Version

2.3

Component

Code

Created by

πŸ‡³πŸ‡±Netherlands batigolix Utrecht

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @batigolix
  • πŸ‡¦πŸ‡ΊAustralia ELC

    It would make more sense to keep the solution and the background together over in ✨ Support external Flood Postponed: needs info , marking this as a duplicate. The code to add Redis (or whatever replaces it) is mostly complete in that issue, it just needs to be merged into the new layout. The code in the patches for FloodUnblockManagerBase in ✨ Make FloodUnblockManager generic Active seem to be based off the original issue. There are a pile of people who deserve credit for getting it to this point, plus more people who would be notified of the updates and changes in plan. I only ran into this due to the posts by Anybody and catch on that issue.

  • Status changed to Needs work 2 months ago
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht

    I announced splitting this issue in this comment:
    https://www.drupal.org/project/flood_control/issues/2928007#comment-1553... ✨ Support external Flood Postponed: needs info

    I would prefer to keep this in a separate issue, to keep the conversation focused on a smaller topic and make it easier to review.

    I ll try to ensure that people are being credited and notified.

  • First commit to issue fork.
  • Assigned to batigolix
  • Status changed to Active 23 days ago
  • πŸ‡³πŸ‡±Netherlands batigolix Utrecht
Production build 0.69.0 2024