CommentFileSizeAuthor
#3 2908307-3.patch7.69 KBnaveenvalecha

Comments

naveenvalecha created an issue. See original summary.

naveenvalecha’s picture

naveenvalecha
He/Him
commented
Title: Move flood to memcache » Implement flood control with memcache
naveenvalecha’s picture

naveenvalecha
He/Him
commented
Status: Active » Needs review
StatusFileSize
new 2908307-3.patch7.69 KB

Here's the start which added a new service but how to clear the cache keys of a specific event. The new service is partially using the Memcache and database so that it gets passed on CI.

rob c’s picture

rob c commented

Great work so far!

I've added a related issue to redis, because this requires a change in flood_unblock to do it right (see the related issue for more details).

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Needs work 
+++ b/src/Flood/MemcacheBackend.php
@@ -0,0 +1,256 @@
+      $this->connection->delete(static::TABLE_NAME)
...
+      $number = $this->connection->select(static::TABLE_NAME, 'f')
...
+      if (!$database_schema->tableExists(static::TABLE_NAME)) {

why you still need db table?