Node-redis: node_redis : ATTENTION : vous avez passé "rediss" comme protocole au lieu du protocole "redis" !

Créé le 23 avr. 2018  ·  22Commentaires  ·  Source: NodeRedis/node-redis

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

cet avertissement s'affiche car rediss: est coché mais rediss serait correct - Ligne 35

pending-author-input

Commentaire le plus utile

Salut tout le monde, j'ai pris la relève en tant que mainteneur principal et j'ai maintenant tous les accès requis 🎉

J'ai passé les derniers jours à préparer le maître pour une version, et il y a quelques minutes, je viens de publier la v3.0.0 sur NPM ; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 - qui inclut ce changement.

Attendez-vous à des publications régulières - ma première priorité pour le moment est de rendre ce projet plus convivial pour les contributeurs afin de m'assurer que le projet vit et continue de croître et qu'il n'est pas bloqué par le temps d'une personne singulière. Pour ce faire, j'aimerais développer un ensemble plus large de contributeurs superficiels. Avec cela, j'espère atténuer le problème précédent du projet nécessitant des mises à jour, mais personne n'a le pouvoir de le faire. Je travaille sur ce qui suit;

  • [x] Contribuer à la documentation et au code de conduite
  • [x] Mettre en place un Open Collective et une politique de dépenses des contributeurs

    • Vous remarquerez le nouveau bouton brillant Sponsor en haut de GitHub, j'ai également pris les devants et l'ai parrainé moi-même et par l'intermédiaire de mon entreprise pour aider à le lancer pour tous les futurs contributeurs

  • WIP : Automatisation des versions et gestion des versions sémantiques (publication sur NPM, génération de journaux des modifications, etc.)
  • [x] Améliorer CI, par exemple Windows CI est super lent et floconneux en ce moment

Après cela, je porterai mon attention sur la modernisation (par exemple, les promesses) et la compensation de la dette technique dans la base de code Node Redis. @BridgeAR a déjà fait un tas de choses pour cela, si vous êtes curieux, consultez la branche WIP v4 et son journal des modifications.

Tous les 22 commentaires

rediss est juste une mauvaise façon parce que le commond fait

Le message devrait en effet être mis à jour, mais il semble que vous ayez en quelque sorte traversé un mauvais protocole.

Pouvez-vous s'il vous plaît montrer ce que vous avez passé à Redis.createClient() en tant qu'arguments (remplacez le nom d'utilisateur / le mot de passe / les URL concrètes) et gardez tout exactement tel qu'il a été passé à node_redis ?

J'ai rencontré le même problème. Ensuite, j'ai réalisé que le commit activant rediss en tant que protocole n'était pas encore publié. La dernière version, 2.8.0 , n'a pas encore cette validation : https://github.com/NodeRedis/node_redis/commit/5d6e471d0c14df39974000427719c5d85fa4f21f

A quand la nouvelle version ? J'ai besoin de cette fonctionnalité.

@BridgeAR @stockholmux @Salakar ping amical. Peut-être qu'une nouvelle version est nécessaire pour cet avertissement.

@Teamop malheureusement, je n'ai pas accès pour publier quoi que ce soit sur npm - seul @BridgeAR le peut.

J'ai rencontré ce problème. Si je regarde le référentiel en ligne, je vois le bloc de déclaration pour rediss.
https://github.com/NodeRedis/node_redis/blob/23ef1e7afadd6022526860d22ac0a7f78495106d/lib/createClient.js#L34

Cependant, lorsque j'examine mon dossier node_modules actuel, la version indique qu'il s'agit de la version 2.8, mais le fichier createClient n'a pas ce bloc d'instructions rediss. J'ai corrigé localement en collant l'instruction rediss if dans mes modules de nœuds locaux et cela a fonctionné. Je ne sais pas en quoi la version distribuée est différente de ce qui est affiché en ligne dans le référentiel.
image

Désolé, je viens de réaliser que la version 2.8.0 a été publiée en 2017, et non en 2018, avant que le support TLS ne soit ajouté à master

@BridgeAR Ce que nous réduisons pour 2.8.0 ne prend pas réellement en charge TLS (c'est-à-dire une URL commençant par "rediss:"). C'est le vrai problème; l'avertissement n'est qu'un symptôme. Voici d'où vient le module node : https://registry.npmjs.org/redis/-/redis-2.8.0.tgz Téléchargez-le et regardez createClient.js. Le code TLS est manquant

Y a-t-il une valeur majeure à utiliser le support TLS ?

J'adore l'avoir aujourd'hui, cela nous permettrait de parler à l'authentification cryptée AWS ElasticCache

@jasonamyers J'ai eu plus de succès avec https://github.com/luin/ioredis sur ce problème - mais vous avez besoin d'une configuration de cluster de cache, avec un cryptage au repos, un cryptage de transport et un mot de passe. Les API sont similaires.

Bonjour, quelqu'un peut-il me conseiller quand cela entre dans le master et est publié ? Ayant ce problème et dû écraser manuellement createClient.js dans node_module. Ce numéro est ouvert depuis plus d'un an. Pouvons-nous pousser cela et autoriser TLS ?

Actuellement, cette section

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!');
}

doivent être remplacés par

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!');
}

dans createClient.js afin d'obtenir le support rediss:// ou vous pouvez essayer une autre méthode comme indiqué ci-dessous.

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

En passant tls comme objet vide, mon client redis s'est connecté sans aucun problème.

@scriptonian cela devrait également résoudre votre problème en évitant l'écrasement manuel dans createClient.js

Veuillez noter :
Dénudez-moi si mon style de commentaire est étrange. Je suis nouveau sur github. :)

merci venkatesh, oui j'avais déjà fait ce changement comme coguy450 l'avait suggéré dans les commentaires. mais je suis surpris que l'équipe n'ait pas mis à jour cela. Cela fait plus d'un an !

+1

y a-t-il une nouvelle version à venir avec ce correctif ?

Quelqu'un veut faire une demande d'extraction pour prendre en charge l'url rediss:// ?
Et vous @venkatesh-nadar 🙂 ?

@peterpoliwoda il semble être déjà ajouté dans la branche master... 👍

Hm... ouais je vois que c'est dans le repo mais npm ne s'installe pas avec le correctif. Le numéro de version n'a pas changé, donc 'npm install' conserve l'ancienne version, comme si elle avait une ancienne archive tar dans l'archive.
Je l'ai fait fonctionner en installant directement depuis Github avec
'''
installation npm https://github.com/NodeRedis/node_redis
'''

Merci beaucoup @venkatesh-nadar !

@BridgeAR Pouvons-nous obtenir une nouvelle version dans npm s'il vous plaît ?

J'ai contacté plusieurs fois @BridgeAR pour demander un accès de publication npm au cours des dernières années et je n'en ai pas entendu parler, je sais qu'il est occupé avec des choses Node.js, ce qui explique probablement pourquoi ce référentiel est négligé.

Discutons de l'avenir du dépôt sur https://github.com/NodeRedis/node_redis/issues/1040#issuecomment -581418899 ?

Salut tout le monde, j'ai pris la relève en tant que mainteneur principal et j'ai maintenant tous les accès requis 🎉

J'ai passé les derniers jours à préparer le maître pour une version, et il y a quelques minutes, je viens de publier la v3.0.0 sur NPM ; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 - qui inclut ce changement.

Attendez-vous à des publications régulières - ma première priorité pour le moment est de rendre ce projet plus convivial pour les contributeurs afin de m'assurer que le projet vit et continue de croître et qu'il n'est pas bloqué par le temps d'une personne singulière. Pour ce faire, j'aimerais développer un ensemble plus large de contributeurs superficiels. Avec cela, j'espère atténuer le problème précédent du projet nécessitant des mises à jour, mais personne n'a le pouvoir de le faire. Je travaille sur ce qui suit;

  • [x] Contribuer à la documentation et au code de conduite
  • [x] Mettre en place un Open Collective et une politique de dépenses des contributeurs

    • Vous remarquerez le nouveau bouton brillant Sponsor en haut de GitHub, j'ai également pris les devants et l'ai parrainé moi-même et par l'intermédiaire de mon entreprise pour aider à le lancer pour tous les futurs contributeurs

  • WIP : Automatisation des versions et gestion des versions sémantiques (publication sur NPM, génération de journaux des modifications, etc.)
  • [x] Améliorer CI, par exemple Windows CI est super lent et floconneux en ce moment

Après cela, je porterai mon attention sur la modernisation (par exemple, les promesses) et la compensation de la dette technique dans la base de code Node Redis. @BridgeAR a déjà fait un tas de choses pour cela, si vous êtes curieux, consultez la branche WIP v4 et son journal des modifications.

Cette page vous a été utile?
0 / 5 - 0 notes

Questions connexes

jackycchen picture jackycchen  ·  4Commentaires

id0Sch picture id0Sch  ·  4Commentaires

strumwolf picture strumwolf  ·  4Commentaires

abhaygarg picture abhaygarg  ·  5Commentaires

michaelwittig picture michaelwittig  ·  3Commentaires