- 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
7 months ago 9:11am 26 April 2024 - π³π±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 infoI 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
5 months ago 11:39am 3 June 2024 - π³π±Netherlands batigolix Utrecht
batigolix β changed the visibility of the branch 2.3.x to hidden.
- Status changed to Needs review
5 months ago 10:32am 28 June 2024 - Issue was unassigned.
- πΊπΈUnited States fskreuz
Did a quick test of this MR with the changes from 3437860. Works for me (Drupal 10.3.1, Flood Control 2.3.4, 3437860's MR, 3437875's MR).
I did see the note about the timestamp and expiration in the MR. But out of curiosity, I checked the entries in Redis. They do seem to have a couple of values that resemble timestamps, and appear to be about 6 hours apart (one's an integer, the other has 4 decimal places). Haven't looked any deeper yet, but at least I can unblock now.
- First commit to issue fork.
- Status changed to Needs work
3 months ago 12:50pm 27 August 2024