[ggivo]

Problem

Jedis.aclLogBinary() / aclLogBinary(int) (declared on AccessControlLogBinaryCommands) declare List<byte[]>, but the ACL LOG reply is a nested array (Array<Array<...>>) — a list of entries, each itself a flattened list of key/value pairs. getBinaryMultiBulkReply() only does an unchecked (List<byte[]>) cast, so the elements are actually nested lists. Reading any element throws ClassCastException: java.util.ArrayList cannot be cast to [B. The methods have never been usable.

Fixes #4579.

Change

• Remove aclLogBinary() and aclLogBinary(int) from Jedis and AccessControlLogBinaryCommands.
• Drop the test-only ACLTestUtil.filterBinaryByClientId and its usage in AccessControlListCommandsTest.
• Document the removal in the v7→v8 migration guide.

Migration

Use aclLog() / aclLog(int), which return structured AccessControlLogEntry objects for the same data.

---

Note

Low Risk
Small API deletion of methods that were never usable; migration path to existing aclLog() APIs is straightforward.

Overview
Removes aclLogBinary() and aclLogBinary(int) from Jedis and AccessControlLogBinaryCommands. Those APIs claimed List<byte[]> but could not parse the nested ACL LOG reply, so they threw ClassCastException at runtime.

Callers should use aclLog() / aclLog(int), which return AccessControlLogEntry objects. The v7→v8 migration guide adds a breaking-change section for this removal. Tests drop binary ACL log coverage and the test-only ACLTestUtil.filterBinaryByClientId helper.

^{Reviewed by Cursor Bugbot for commit c190541. Bugbot is set up for automated code reviews on this repo. Configure here.}