ENH Only query primary DB

silverstripe · Sep 24, 2024 · 16e754e · 16e754e
1 parent bba7c4f
commit 16e754e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/composer.json b/composer.json
@@ -16,7 +16,7 @@
     ],
     "require": {
         "php": "^8.1",
-        "silverstripe/framework": "^5"
+        "silverstripe/framework": "^5.4"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.6",

diff --git a/src/Model/HybridSessionDataObject.php b/src/Model/HybridSessionDataObject.php
@@ -20,4 +20,9 @@ class HybridSessionDataObject extends DataObject
     ];
 
     private static $table_name = 'HybridSessionDataObject';
+
+    /**
+     * Only use the primary db to ensure there is no risk of reading from an unsynced replica db
+     */
+    private static bool $must_use_primary_db = true;
 }
diff --git a/src/Store/DatabaseStore.php b/src/Store/DatabaseStore.php
@@ -68,7 +68,7 @@ public function read(string $session_id): string|false
             $this->getNow()
         );
 
-        $result = DB::query($query);
+        $result = DB::withPrimary(fn() => DB::query($query));
 
         if ($result && $result->numRecords()) {
             $data = $result->record();