Files

chrome-storm-c442 997c7d20a5 v1.8.66: fix 7 Redis bugs — shlex parser, GUI stats, SSL, error handling, dedup

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

2026-02-28 07:15:46 -05:00

8.3 KiB

Raw Blame History

Аудит Redis — полный отчёт

Дата: 2026-02-28 Версия: v1.8.65 Тестовый сервер: Reddis main ovh (Redis 8.2.1, 27M+ ключей, 2.82GB RAM)

1. Живое тестирование через CLI

Работает корректно

Команда	Результат
`--redis-info ALIAS`	Версия, память, клиенты, keyspace
`--redis-keys ALIAS "*"`	SCAN до 1000 ключей, сортировка
`--redis ALIAS "PING"`	`True`
`--redis ALIAS "DBSIZE"`	`27199166`
`--redis ALIAS "GET key"`	Возвращает значение
`--redis ALIAS "SET key value"`	`True`
`--redis ALIAS "DEL key1 key2"`	Число удалённых
`--redis ALIAS "KEYS pattern"`	Список с нумерацией
`--redis ALIAS "INFO memory"`	Полный вывод секции
`--redis ALIAS "TYPE key"`	Тип данных
`--redis ALIAS "TTL key"`	Секунды / -1
`--redis ALIAS "SELECT 1"`	`True`
`--redis ALIAS "MSET k1 v1 k2 v2"`	`True`
`--status` (redis)	`ONLINE`

Сломано

Команда	Ошибка	Баг
`SET key "hello world"`	`syntax error`	#1
`SET key '{"a":1, "b":2}'`	`syntax error`	#1
Любое значение с пробелами	`syntax error`	#1

2. Найденные баги

БАГ #1 — КРИТИЧНЫЙ: Парсер команд ломает значения с пробелами

Файлы: tools/ssh.py:958, core/redis_client.py:78

Проблема: Используется command.split() вместо shlex.split().

# Текущий код:
parts = command.split()
# "SET key 'hello world'" → ['SET', 'key', "'hello", "world'"]
# Redis получает 4 аргумента вместо 3 → syntax error

Воздействие:

Невозможно SET/GET значения с пробелами
Невозможно работать с JSON-данными содержащими пробелы
Невозможно выполнить EVAL с Lua-скриптами
Сломаны команды с multi-word аргументами (SORT, CONFIG SET и др.)

Затронуты: CLI (tools/ssh.py) И GUI (core/redis_client.py → redis_tab.py)

БАГ #2 — СРЕДНИЙ: GUI stats показывает отформатированную строку

Файл: gui/tabs/redis_tab.py:192-208

Проблема: _refresh_stats() вызывает client.execute("DBSIZE"), который возвращает строку "(integer) 27199166" (уже отформатированную через _format()). Эта строка отображается как есть.

# redis_tab.py:198
keys_count = client.execute("DBSIZE")    # Returns "(integer) 27199166"
keys_text = str(keys_count)              # → "(integer) 27199166" in label

Для INFO memory проблема другая:

# redis_tab.py:201
info = client.execute("INFO memory")
# _format() превращает dict в строку, но без \r\n
# Поиск по \r\n ничего не найдёт → memory = "—" всегда
for line in info.split("\r\n"):           # \r\n не найдётся!
    if line.startswith("used_memory_human:"):
        memory = line.split(":")[1].strip()

Воздействие: Статистика в GUI:

Keys показывает (integer) 27199166 вместо 27199166
Memory всегда показывает — (парсинг не находит \r\n)

БАГ #3 — СРЕДНИЙ: GUI RedisClient не поддерживает SSL

Файл: core/redis_client.py:35-43

Проблема: В конструкторе redis.Redis() отсутствует параметр ssl.

# redis_client.py:35-43
self._conn = r.Redis(
    host=self._host,
    port=self._port,
    password=self._password,
    db=self._db,
    decode_responses=True,
    socket_timeout=5,
    socket_connect_timeout=5,
    # ❌ ОТСУТСТВУЕТ: ssl=...
)

При этом CLI (tools/ssh.py:955-956) SSL поддерживает:

ssl_enabled = server.get("ssl", False)
r = redis_lib.Redis(..., ssl=ssl_enabled)

Воздействие: GUI не может подключиться к Redis с TLS/SSL.

Связанная проблема: В server_dialog.py поле use_ssl/ssl НЕ включено в FIELD_MAP["redis"]:

"redis": ["password", "db_index"]   # нет ssl!

Даже если добавить SSL в RedisClient, пользователь не сможет включить его в UI.

БАГ #4 — НИЗКИЙ: CLI Redis ошибки выдают сырой traceback

Файл: tools/ssh.py:957-962

Проблема: r.execute_command() не обёрнут в try-except.

# Текущий код:
result = r.execute_command(*parts)   # При ошибке → необработанное исключение

Пример: HGETALL на string-ключе:

ERROR: ResponseError: WRONGTYPE Operation against a key holding the wrong kind of value

Выводится как traceback с двойным повтором вместо читаемого сообщения.

БАГ #5 — НИЗКИЙ: Двойной PING при status check

Файл: core/status_checker.py:98-110

def _check_redis(self, server: dict) -> bool:
    client = RedisClient(server)
    result = client.connect()      # ← PING #1 внутри connect()
    if result:
        ok = client.check_connection()  # ← PING #2
        client.disconnect()
        return ok
    return False

connect() уже делает self._conn.ping() → True. Потом check_connection() вызывает self._conn.ping() ещё раз.

БАГ #6 — НИЗКИЙ: CLI status для Redis без try-except

Файл: tools/ssh.py:698-703

if stype == "redis":
    r = redis_lib.Redis(...)
    r.ping()        # ← Нет try-except! Если offline → traceback
    r.close()
    return "ONLINE"

Сравни с SQL, где conn.close() в finally. Здесь если ping() упадёт, r.close() не вызовется.

ПРОБЛЕМА #7 — Дублирование кода подключения

Файлы: tools/ssh.py:946-956, tools/ssh.py:978-986, tools/ssh.py:1009-1017

Три функции (run_redis_cmd, redis_info, redis_keys) создают одинаковое подключение к Redis копипастом:

import redis as redis_lib
host = server["ip"]
port = server.get("port", 6379)
password = server.get("password", "") or None
db_index = server.get("db_index", 0)
ssl_enabled = server.get("ssl", False)
r = redis_lib.Redis(host=host, port=port, ...)

Этот блок повторяется 3 раза. Вынести в хелпер.

3. Карта проблем по файлам

Файл	Строки	Баги
`tools/ssh.py`	958	#1 (split), #4 (error handling), #6 (status), #7 (copypaste)
`core/redis_client.py`	78, 35-43	#1 (split), #3 (no SSL)
`gui/tabs/redis_tab.py`	192-208	#2 (stats parsing)
`core/status_checker.py`	98-110	#5 (double ping)
`gui/server_dialog.py`	22	#3 (no ssl field for redis)

4. Влияние на пользователей

Сценарий	Статус	Причина
Простые команды (GET/SET/DEL)	✅ Работает	Нет пробелов
Команды со значениями с пробелами	❌ Сломано	Баг #1
JSON-данные с пробелами	❌ Сломано	Баг #1
GUI статистика Keys	⚠️ Некорректно	Баг #2
GUI статистика Memory	❌ Всегда "—"	Баг #2
SSL-подключение через GUI	❌ Невозможно	Баг #3
SSL-подключение через CLI	✅ Работает	—
Ошибки в CLI	⚠️ Нечитаемые	Баг #4
Status check	✅ Работает (с оверхедом)	Баг #5

8.3 KiB Raw Blame History Unescape Escape