1
0
mirror of https://github.com/redis/go-redis.git synced 2026-10-24 07:59:20 +03:00

1 Commits

Author SHA1 Message Date
sadd amr3e
dbedca714c fix(command): reconcile declared entry-array length in reply parsers (#3907)
* reconcile declared entry-array length in reply parsers

* drain extra GEOSEARCH entry elements instead of erroring

* fix(command): repair SLOWLOG GET merge resolution

The conflict resolution in b895673c kept master's nn > 7 upper-bound
error alongside the drain loop (making the drain dead code, against the
PR's intent) and broke the brace structure, so the package no longer
compiled. Keep the Redis 8.10+ CommandArgc field from master, drop the
upper-bound error, and drain any elements past the 7 known fields.

* test(command): exercise SlowLog drain loop

Bugbot flagged that a 7-element entry is fully consumed by the parser
now that CommandArgc is the 7th known field, so the drain loop never
ran. Declare 8 elements — the 7 known fields plus one extra — and
assert CommandArgc is parsed rather than drained.

---------

Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com>
Co-authored-by: Nedyalko Dyakov <nedyalko.dyakov@gmail.com>
2026-07-29 13:36:18 +03:00