open-duelyst/duelyst

[P3] Replace node-redis with ioredis

Offen

#80 geöffnet am 02.10.2022

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (526 Forks)batch import
backendenhancementhelp wanted

Repository-Metriken

Stars
 (3.443 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

We're currently using node-redis@2.8.0, which has poor support for connection timeouts.

Replacing this with ioredis@5.2.3 fixes the above, but requires some changes. For example, the zrevrankAsync function does not exist, so we need to use zrevrank instead and remove the .then(result) chain.

Alternatively, upgrading node-redis may help with these issues.

See https://github.com/luin/ioredis/wiki/Migrating-from-node_redis

Contributor Guide

Research-Richtung
Überprüfe den Migrationsleitfaden unter https://github.com/luin/ioredis/wiki/Migrating from node redis. Ersetze node redis durch ioredis im Codebase, insbesondere aktualisiere Methoden wie `zrevrankAsync` zu `zrevrank` und entferne `.then()` Ketten. Überprüfe alle Redis Aufrufe, um Kompatibilität sicherzustellen.
Tech Stack
javascriptnodejsredis
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptNode.jsRedis
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60