Skip to content

Commit

Permalink
Merge pull request #50113 from nextcloud/setupChecksMoveFromBinary
Browse files Browse the repository at this point in the history
fix(SetupChecks): Don't test caches using binary data
  • Loading branch information
solracsf authored Jan 10, 2025
2 parents b51513f + cd147cd commit 3fea444
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions apps/settings/lib/SetupChecks/MemcacheConfigured.php
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ public function run(): SetupResult {
}

if ($this->cacheFactory->isLocalCacheAvailable()) {
$random = random_bytes(64);
$random = bin2hex(random_bytes(64));
$local = $this->cacheFactory->createLocal('setupcheck.local');
try {
$local->set('test', $random);
Expand All @@ -77,7 +77,7 @@ public function run(): SetupResult {
}

if ($this->cacheFactory->isAvailable()) {
$random = random_bytes(64);
$random = bin2hex(random_bytes(64));
$distributed = $this->cacheFactory->createDistributed('setupcheck');
try {
$distributed->set('test', $random);
Expand Down

0 comments on commit 3fea444

Please sign in to comment.