Skip to content

Commit b72182d

Browse files
authored
Merge pull request #5 from syzygypl/bugfix/session-handler-must-return-boolean
Redis session handler methods expect boolean result.
2 parents 2ff31bc + bc652fc commit b72182d

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/Session/RedisSessionHandler.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,19 +30,23 @@ public function __construct(\Redis $redis, $keyPrefix = 'session:')
3030

3131
public function open($savePath, $name)
3232
{
33+
return true;
3334
}
3435

3536
public function gc($maxLifetime)
3637
{
38+
return true;
3739
}
3840

3941
public function close()
4042
{
43+
return true;
4144
}
4245

4346
public function destroy($sessionId)
4447
{
4548
$this->redis->del($this->keyPrefix . $sessionId);
49+
return true;
4650
}
4751

4852
/**
@@ -68,5 +72,7 @@ public function write($sessionId, $sessionData)
6872
$this->redis->set($sessionId, $sessionData);
6973

7074
$this->redis->expire($sessionId, $this->maxLifetime);
75+
76+
return true;
7177
}
7278
}

0 commit comments

Comments
 (0)