From cd123987076fddcb6cbd4a90fbcff5675f82b563 Mon Sep 17 00:00:00 2001
From: Evgeny Zinoviev <me@ch1p.io>
Date: Sun, 16 May 2021 02:43:13 +0300
Subject: server/cache: fix cache collision for commands with different
 arguments

---
 src/server/server.cc | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

(limited to 'src/server/server.cc')

diff --git a/src/server/server.cc b/src/server/server.cc
index ab1db9e..2c2ff33 100644
--- a/src/server/server.cc
+++ b/src/server/server.cc
@@ -114,7 +114,7 @@ std::shared_ptr<p18::response_type::BaseResponse> Server::executeCommand(p18::Co
     auto it = cache_.find(commandType);
     if (it != cache_.end()) {
         auto cr = it->second;
-        if (voltronic::timestamp() - cr.time <= cacheTimeout_) {
+        if (voltronic::timestamp() - cr.time <= cacheTimeout_ && arguments == cr.arguments) {
             return cr.response;
         }
 
@@ -123,7 +123,11 @@ std::shared_ptr<p18::response_type::BaseResponse> Server::executeCommand(p18::Co
 
     try {
         auto response = client_.execute(commandType, arguments);
-        CachedResponse cr{voltronic::timestamp(), response};
+        CachedResponse cr {
+            .time = voltronic::timestamp(),
+            .arguments = arguments,
+            .response = response
+        };
         cache_[commandType] = cr;
         return response;
     }
@@ -141,5 +145,4 @@ std::shared_ptr<p18::response_type::BaseResponse> Server::executeCommand(p18::Co
     }
 }
 
-
 }
\ No newline at end of file
-- 
cgit v1.2.3