Fix reset! when using namespaced cache store

Author: santib

lib/rack/attack/cache.rb

@@ -55,7 +55,7 @@ def delete(unprefixed_key)
 
       def reset!
         if store.respond_to?(:delete_matched)
-          store.delete_matched("#{prefix}*")
+          store.delete_matched(/#{prefix}*/)
         else
           raise(
             Rack::Attack::IncompatibleStoreError,

lib/rack/attack/store_proxy/redis_proxy.rb

@@ -49,7 +49,7 @@ def delete_matched(matcher, _options = nil)
           rescuing do
             # Fetch keys in batches using SCAN to avoid blocking the Redis server.
             loop do
-              cursor, keys = scan(cursor, match: matcher, count: 1000)
+              cursor, keys = scan(cursor, match: matcher.source, count: 1000)
               del(*keys) unless keys.empty?
               break if cursor == "0"
             end

spec/rack_attack_spec.rb

@@ -124,5 +124,18 @@
         _(redis.get("#{Rack::Attack.cache.prefix}::key")).must_be_nil
       end
     end
+
+    if defined?(ActiveSupport::Cache::MemoryStore)
+      it 'should delete rack attack keys' do
+        memory_store = ActiveSupport::Cache::MemoryStore.new(namespace: 'ns')
+        memory_store.write('key', 'value')
+        memory_store.write("#{Rack::Attack.cache.prefix}::key", 'value')
+        Rack::Attack.cache.store = memory_store
+        Rack::Attack.reset!
+
+        _(memory_store.read('key')).must_equal 'value'
+        _(memory_store.read("#{Rack::Attack.cache.prefix}::key")).must_be_nil
+      end
+    end
   end
 end