mirror of
https://github.com/predis/predis.git
synced 2026-08-29 11:41:29 +00:00
Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 675951360f | |||
| 0795d69d9e |
@@ -1,5 +1,11 @@
|
||||
## Changelog
|
||||
|
||||
## Unlreleads
|
||||
### Added
|
||||
### Changed
|
||||
### Fixed
|
||||
- Fixed RESP3 double parsing returning positive `INF` for `-inf` payloads (#1716)
|
||||
|
||||
## v3.6.0 (2026-08-14)
|
||||
### Added
|
||||
- Added support for new TS commands + Indonesian language support integration test (#1695)
|
||||
|
||||
+1
-1
@@ -56,7 +56,7 @@ use Traversable;
|
||||
*/
|
||||
class Client implements ClientInterface, IteratorAggregate
|
||||
{
|
||||
public const VERSION = '3.6.0';
|
||||
public const VERSION = '3.6.1-dev';
|
||||
|
||||
/** @var OptionsInterface */
|
||||
private $options;
|
||||
|
||||
@@ -63,10 +63,14 @@ class Resp3Strategy extends Resp2Strategy
|
||||
*/
|
||||
protected function parseDouble(string $string): float
|
||||
{
|
||||
if ($string === 'inf' || $string === '-inf') {
|
||||
if ($string === 'inf') {
|
||||
return INF;
|
||||
}
|
||||
|
||||
if ($string === '-inf') {
|
||||
return -INF;
|
||||
}
|
||||
|
||||
return (float) $string;
|
||||
}
|
||||
|
||||
|
||||
@@ -60,11 +60,11 @@ class Resp3StrategyTest extends PredisTestCase
|
||||
* @param string $data
|
||||
* @return void
|
||||
*/
|
||||
public function testParseDataReturnsFloatInfinityOnInfinityOrNegativeInfinity(string $data): void
|
||||
public function testParseDataReturnsFloatInfinityOnInfinityOrNegativeInfinity(string $data, float $expectedValue): void
|
||||
{
|
||||
$actualResponse = $this->strategy->parseData($data);
|
||||
|
||||
$this->assertInfinite($actualResponse);
|
||||
$this->assertSame($expectedValue, $actualResponse);
|
||||
}
|
||||
|
||||
/**
|
||||
@@ -166,8 +166,8 @@ class Resp3StrategyTest extends PredisTestCase
|
||||
public function infinityProvider(): array
|
||||
{
|
||||
return [
|
||||
'positive infinity' => [",inf\r\n"],
|
||||
'negative infinity' => [",-inf\r\n"],
|
||||
'positive infinity' => [",inf\r\n", INF],
|
||||
'negative infinity' => [",-inf\r\n", -INF],
|
||||
];
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user