Node-redis: support tls sur l'url redis

Je suis contre l'ajout de ceci. La raison en est qu'il n'est pas possible d'ajouter toutes les options tls avec l'url. Par conséquent, davantage d'options devraient être transmises au client dans la plupart des cas. Il est également difficile de distinguer les options tls options régulières. Merci d'avoir attiré mon attention sur ce point !

Mongodb par exemple le gère... https://docs.mongodb.com/manual/reference/connection-string/
De plus, compte tenu du fait que vous acceptez déjà de ne pas couvrir toutes les options via l'url, je ne vois pas l'intérêt de ne pas ajouter plus de support.

Je ne vois pas pourquoi il est nécessaire "d'ajouter toutes les options tls avec l'url". Du point de vue des opérations, il suffirait d'activer le cryptage TLS pour la connexion - l'établissement de la relation de confiance doit être géré par le système d'exploitation de toute façon (autorités de certification racine/intermédiaire installées ; voir /etc/ssl sur Linux). Je ne veux vraiment pas approfondir les paramètres de chaque langage de programmation afin de "juste activer TLS". Cela devrait fondamentalement être une chose oui/non. C'est ce que fournit le schéma d'uri rediss .

D'autres bibliothèques clientes (dans d'autres langues) prennent déjà en charge ceci :

• https://github.com/nrk/predis (de plus, les DSN comme "tls://127.0.0.1?ssl[cafile]=private.pem&ssl[verify_peer]=1" sont pris en charge)
• https://github.com/andymccurdy/redis-py

En ne fournissant pas cette fonctionnalité, il est nécessaire de conserver une configuration distincte pour node.js.

On dirait qu'il y avait suffisamment d'intérêt pour cette fonctionnalité pour que je puisse reconsidérer cela.

Une version NPM pour cette fonctionnalité est-elle prévue prochainement ? 2.8.0 est la dernière version, qui n'a pas cette fonctionnalité.

Qu'en est-il d'une forme moindre de cela qui fait quelque chose comme définir les options tls par défaut qu'il _peut_ définir à partir de l'URL sans étendre le format ? La définition de servername sur la partie hostname de l'URL, par exemple, peut être une chose raisonnablement sûre à faire.

Je ne sais pas pour les autres, mais je sais qu'au moins compose.io utilise SNI pour Redis sur SSL .

Cela a été partiellement résolu via https://github.com/NodeRedis/node_redis/pull/1282

@BridgeAR Il reste inédit, un espoir de sortie prochainement ?

Je viens de publier la v3.0.0 sur NPM ; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 - qui inclut le changement @calebboyd mentionné, ce problème peut-il être clos maintenant ?