Node-redis: node_redis: WARNUNG: Sie haben "rediss" als Protokoll anstelle des "redis"-Protokolls übergeben!

Erstellt am 23. Apr. 2018  ·  22Kommentare  ·  Quelle: NodeRedis/node-redis

node_redis: WARNING: You passed "rediss" as protocol instead of the "redis" protocol!

diese Warnung wird angezeigt, weil rediss: markiert ist, aber rediss korrekt wäre - Zeile 35

pending-author-input

Hilfreichster Kommentar

Hallo zusammen, ich habe die Rolle des Hauptbetreuers übernommen und habe jetzt alle erforderlichen Zugriffe 🎉

Ich habe die letzten paar Tage damit verbracht, Master für eine Veröffentlichung vorzubereiten, und vor ein paar Minuten habe ich gerade v3.0.0 für NPM veröffentlicht; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 – was diese Änderung enthält.

Erwarten Sie regelmäßige Veröffentlichungen - meine erste Priorität ist es im Moment, dieses Projekt beitragsfreundlicher zu machen, um sicherzustellen, dass das Projekt lebt und weiter wächst und nicht durch die Zeit einer einzelnen Person blockiert wird. Um dies zu tun, möchte ich eine größere Gruppe von flachen Mitwirkenden anbauen. Damit hoffe ich, das vorherige Problem zu entschärfen, dass das Projekt Updates benötigt, aber es gibt niemanden, der die Macht dazu hat. Ich arbeite folgendes dazu durch;

  • [x] Beitragende Dokumente & Verhaltenskodex
  • [x] Richten Sie eine Offene Kollektiv- und eine Spender- Spesenrichtlinie ein

    • Sie werden den glänzenden neuen Sponsor -Button oben auf GitHub bemerken, ich habe ihn auch selbst und durch mein Unternehmen gesponsert, um ihn für alle zukünftigen Mitwirkenden in Gang zu bringen

  • WIP: Release-Automatisierung und semantische Versionierung (Veröffentlichung in NPM, Generierung von Änderungsprotokollen usw.)
  • [x] CI verbessern, z. B. Windows CI ist momentan super langsam und unzuverlässig

Danach werde ich meine Aufmerksamkeit auf die Modernisierung (z. B. Versprechungen) und die Beseitigung technischer Schulden in der Node Redis-Codebasis richten. @BridgeAR hat dafür bereits eine Menge Sachen gemacht, wenn Sie neugierig sind, sehen Sie sich den WIP v4-Zweig und sein Änderungsprotokoll an.

Alle 22 Kommentare

Rediss ist einfach der falsche Weg, weil die Commond es tun

Die Nachricht sollte tatsächlich aktualisiert werden, aber es scheint, als ob Sie irgendwie ein falsches Protokoll durchlaufen haben.

Können Sie bitte zeigen, was Sie als Argumente an Redis.createClient() übergeben haben (Benutzername / Passwort / konkrete URLs ersetzen) und alles genau so beibehalten, wie es an node_redis übergeben wurde?

Ich bin auf das gleiche Problem gestoßen. Dann wurde mir klar, dass das Commit, das rediss als Protokoll aktiviert, noch nicht veröffentlicht wurde. Die neueste Version, 2.8.0 , hat diesen Commit noch nicht: https://github.com/NodeRedis/node_redis/commit/5d6e471d0c14df39974000427719c5d85fa4f21f

Wann kommt die Neuerscheinung? Ich brauche diese Funktion.

@BridgeAR @stockholmux @Salakar freundlicher Ping. Möglicherweise ist für diese Warnung eine neue Version erforderlich.

@Teamop leider habe ich keinen Zugriff, um etwas auf npm zu veröffentlichen - nur @BridgeAR kann.

Ich bin auf dieses Problem gestoßen. Wenn ich mir das Online-Repo ansehe, sehe ich den Anweisungsblock für Rediss.
https://github.com/NodeRedis/node_redis/blob/23ef1e7afadd6022526860d22ac0a7f78495106d/lib/createClient.js#L34

Wenn ich jedoch meinen tatsächlichen node_modules-Ordner untersuche, sagt die Version, dass es 2.8 ist, aber die createClient-Datei hat diesen Rediss-Anweisungsblock nicht. Ich habe es lokal behoben, indem ich die rediss if-Anweisung in meine lokalen Knotenmodule eingefügt habe, und das hat funktioniert. Ich bin mir nicht sicher, wie sich die verteilte Version von dem unterscheidet, was online im Repo angezeigt wird.
image

Tut mir leid, ich habe gerade festgestellt, dass 2.8.0 2017 veröffentlicht wurde, nicht 2018, bevor die TLS-Unterstützung zu Master hinzugefügt wurde

@BridgeAR Was wir für 2.8.0 herunterziehen, unterstützt TLS nicht wirklich (dh eine URL, die mit "rediss:") beginnt. Das ist das eigentliche Problem; Die Warnung ist nur ein Symptom. Hier ist, woher das Knotenmodul stammt: https://registry.npmjs.org/redis/-/redis-2.8.0.tgz Laden Sie es herunter und sehen Sie sich createClient.js an. Der TLS-Code fehlt

Hat die Verwendung der TLS-Unterstützung einen größeren Wert?

Ich hätte es heute gerne, es würde uns ermöglichen, mit AWS ElasticCache mit verschlüsselter Authentifizierung zu sprechen

@jasonamyers Ich hatte mehr Erfolg mit https://github.com/luin/ioredis zu diesem Thema - aber Sie benötigen ein Cachecluster-Setup mit Verschlüsselung im Ruhezustand, Transportverschlüsselung und einem Passwort. APIs sind ähnlich.

Hallo, kann mir bitte jemand sagen, wann das in den Master geht und veröffentlicht wird? Hatte dieses Problem und musste createClient.js in node_module manuell überschreiben. Diese Ausgabe ist seit über einem Jahr geöffnet. Können wir das verdrängen und TLS zulassen?

Derzeit dieser Abschnitt

if (parsed.protocol && parsed.protocol !== 'redis:') {
    console.warn('node_redis: WARNING: You passed "' + parsed.protocol.substring(0, parsed.protocol.length - 1) + '" as protocol instead of the "redis" protocol!');
}

ersetzt werden müssen durch

if (parsed.protocol === 'rediss:') {
    options.tls = options.tls || {};
} else if (parsed.protocol !== 'redis:') {
    console.warn('node_redis: WARNING: You passed "' + parsed.protocol.substring(0, parsed.protocol.length - 1) + '" as protocol instead of the "redis" protocol!');
}

in createClient.js, um rediss://- Unterstützung zu erhalten, oder Sie könnten einen anderen Weg versuchen, wie unten gezeigt.

var redis = require("redis");
var client = redis.createClient("redis://:password<strong i="13">@host</strong>:port", {tls:{}});

Durch das Übergeben von tls als leeres Objekt wurde mein Redis-Client ohne Probleme verbunden.

@scriptonian Dies sollte auch Ihr Problem lösen, indem Sie das manuelle Überschreiben in createClient.js vermeiden

Bitte beachten Sie :
Entblößen Sie mich, wenn mein Kommentarstil seltsam ist. ich bin neu bei github. :)

danke venkatesh, ja, ich hatte diese änderung bereits vorgenommen, wie coguy450 es in den kommentaren vorgeschlagen hatte. aber ich bin überrascht, dass das Team dies nicht aktualisiert hat. Es ist über ein Jahr her!

+1

Kommt mit diesem Fix eine neue Version?

Möchte jemand einen Pull-Request stellen, um die rediss://-URL zu unterstützen?
Wie wäre es mit dir @venkatesh-nadar 🙂 ?

@peterpoliwoda es scheint bereits im Master-Zweig hinzugefügt worden zu sein... 👍

Hm ... ja, ich sehe, es ist im Repo, aber npm wird nicht mit dem Fix installiert. Die Versionsnummer hat sich nicht geändert, also behält 'npm install' den alten Build bei, als hätte es einen alten Tarball im Archiv.
Ich habe es zum Laufen gebracht, indem ich direkt von Github mit installiert habe
'''
npm installieren https://github.com/NodeRedis/node_redis
'''

Tausend Dank @venkatesh-nadar!

@BridgeAR Können wir bitte eine neue Version in npm bekommen?

Ich habe mich in den letzten Jahren mehrmals an @BridgeAR gewandt , um npm-Veröffentlichungszugriff anzufordern, und nichts davon gehört. Ich weiß, dass er mit Node.js-Dingen beschäftigt ist, weshalb dieses Repo wahrscheinlich vernachlässigt wird.

Lassen Sie uns die Repo-Zukunft unter https://github.com/NodeRedis/node_redis/issues/1040#issuecomment -581418899 besprechen?

Hallo zusammen, ich habe die Rolle des Hauptbetreuers übernommen und habe jetzt alle erforderlichen Zugriffe 🎉

Ich habe die letzten paar Tage damit verbracht, Master für eine Veröffentlichung vorzubereiten, und vor ein paar Minuten habe ich gerade v3.0.0 für NPM veröffentlicht; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 – was diese Änderung enthält.

Erwarten Sie regelmäßige Veröffentlichungen - meine erste Priorität ist es im Moment, dieses Projekt beitragsfreundlicher zu machen, um sicherzustellen, dass das Projekt lebt und weiter wächst und nicht durch die Zeit einer einzelnen Person blockiert wird. Um dies zu tun, möchte ich eine größere Gruppe von flachen Mitwirkenden anbauen. Damit hoffe ich, das vorherige Problem zu entschärfen, dass das Projekt Updates benötigt, aber es gibt niemanden, der die Macht dazu hat. Ich arbeite folgendes dazu durch;

  • [x] Beitragende Dokumente & Verhaltenskodex
  • [x] Richten Sie eine Offene Kollektiv- und eine Spender- Spesenrichtlinie ein

    • Sie werden den glänzenden neuen Sponsor -Button oben auf GitHub bemerken, ich habe ihn auch selbst und durch mein Unternehmen gesponsert, um ihn für alle zukünftigen Mitwirkenden in Gang zu bringen

  • WIP: Release-Automatisierung und semantische Versionierung (Veröffentlichung in NPM, Generierung von Änderungsprotokollen usw.)
  • [x] CI verbessern, z. B. Windows CI ist momentan super langsam und unzuverlässig

Danach werde ich meine Aufmerksamkeit auf die Modernisierung (z. B. Versprechungen) und die Beseitigung technischer Schulden in der Node Redis-Codebasis richten. @BridgeAR hat dafür bereits eine Menge Sachen gemacht, wenn Sie neugierig sind, sehen Sie sich den WIP v4-Zweig und sein Änderungsprotokoll an.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

strumwolf picture strumwolf  ·  4Kommentare

Atala picture Atala  ·  3Kommentare

lemon707 picture lemon707  ·  3Kommentare

Mickael-van-der-Beek picture Mickael-van-der-Beek  ·  6Kommentare

betimer picture betimer  ·  5Kommentare