Node-redis: node_redis: ПРЕДУПРЕЖДЕНИЕ: вы передали «redis» в качестве протокола вместо протокола «redis»!

Созданный на 23 апр. 2018  ·  22Комментарии  ·  Источник: NodeRedis/node-redis

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

это предупреждение отображается, потому что rediss: проверяется, но rediss будет правильным — строка 35

pending-author-input

Самый полезный комментарий

Всем привет, я стал ведущим сопровождающим и теперь имею все необходимые права доступа 🎉

Последние несколько дней я готовил master к выпуску, и несколько минут назад я только что опубликовал v3.0.0 в NPM; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 , который включает это изменение.

Ожидайте регулярных релизов — мой главный приоритет на данный момент — сделать этот проект более удобным для участников, чтобы гарантировать, что проект живет и продолжает расти и не блокируется временем какого-либо отдельного человека. Для этого я хотел бы вырастить большее количество мелких участников. С этим я надеюсь смягчить предыдущую проблему проекта, нуждающегося в обновлениях, но нет никого, кто мог бы это сделать. Я работаю над этим;

  • [x] Предоставление документов и Кодекс поведения
  • [x] Настройка открытого коллектива и политики расходов участников

    • Вы заметите новую блестящую кнопку Sponsor в верхней части GitHub, я также пошел дальше и спонсировал ее сам и через свою компанию, чтобы помочь запустить ее для любых будущих участников.

  • WIP: автоматизация выпуска и семантическое управление версиями (публикация в NPM, создание журналов изменений и т. д.)
  • [x] Улучшить CI, например, Windows CI сейчас очень медленный и ненадежный

После этого я переключу свое внимание на модернизацию (например, промисы) и устранение технического долга в кодовой базе Node Redis. @BridgeAR уже сделал для этого кучу всего, если вам интересно, посмотрите ветку WIP v4 и ее журнал изменений.

Все 22 Комментарий

rediss - это просто неправильный путь, потому что commond делает

Сообщение действительно должно быть обновлено, но похоже, что вы каким-то образом прошли через неправильный протокол.

Не могли бы вы показать, что вы передали в Redis.createClient() в качестве аргументов (замените имя пользователя/пароль/конкретные URL-адреса) и сохраните все точно так, как оно было передано в node_redis?

Я столкнулся с той же проблемой. Затем я понял, что фиксация, включающая rediss , поскольку протокол еще не выпущен. Последняя версия 2.8.0 еще не имеет этой фиксации: https://github.com/NodeRedis/node_redis/commit/5d6e471d0c14df39974000427719c5d85fa4f21f

Когда новый выпуск? Мне нужна эта функция.

@BridgeAR @stockholmux @Salakar дружественный пинг. Возможно, для этого предупреждения необходим новый релиз.

@Teamop , к сожалению, у меня нет доступа для публикации чего-либо в npm — это может сделать только @BridgeAR .

Я столкнулся с этой проблемой. Если посмотреть на онлайн-репозиторий, я вижу блок операторов для повторения.
https://github.com/NodeRedis/node_redis/blob/23ef1e7afadd6022526860d22ac0a7f78495106d/lib/createClient.js#L34

Однако, когда я просматриваю свою фактическую папку node_modules, версия говорит, что это 2.8, однако в файле createClient нет этого блока оператора rediss. Я исправил локально, вставив оператор rediss if в мои локальные модули узла, и это сработало. Я не уверен, чем распределенная версия отличается от того, что показано в репозитории онлайн.
image

Извините, я только что понял, что 2.8.0 была выпущена в 2017 году, а не в 2018 году, до того, как в master была добавлена ​​​​поддержка TLS

@BridgeAR То, что мы собираем для 2.8.0, на самом деле не поддерживает TLS (то есть URL-адрес, начинающийся с «redis:»). Это настоящая проблема; предупреждение - это всего лишь симптом. Вот откуда взялся модуль node: https://registry.npmjs.org/redis/-/redis-2.8.0.tgz Загрузите его и посмотрите на createClient.js. Код TLS отсутствует

Есть ли какая-то большая ценность в использовании поддержки TLS?

Мне нравится иметь его сегодня, это позволит нам использовать зашифрованную аутентификацию AWS ElasticCache.

@jasonamyers Я добился большего успеха в этом вопросе с https://github.com/luin/ioredis , но вам нужна настройка cachecluster с шифрованием в состоянии покоя, транспортным шифрованием и паролем. API похожи.

Здравствуйте, может кто-нибудь посоветовать, когда это будет в мастере и будет выпущено? Возникла эта проблема, и пришлось вручную перезаписать createClient.js в node_module. Этот вопрос открыт уже больше года. Можем ли мы вытолкнуть это и разрешить TLS?

В настоящее время этот раздел

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

необходимо заменить на

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

в createClient.js, чтобы получить поддержку rediss:// , или вы можете попробовать другой способ, как показано ниже.

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

Передав tls как пустой объект, мой клиент Redis подключился без проблем.

@scriptonian это также должно решить вашу проблему, избегая ручной перезаписи в createClient.js

Пожалуйста, обрати внимание :
Обнажите меня, если мой стиль комментирования странный. Я новичок в гитхабе. :)

спасибо, venkatesh, да, я уже внес это изменение, поскольку coguy450 предложил это в комментариях. но я удивлен, что команда не обновила это. Прошло больше года!

+1

есть ли новая версия с этим исправлением?

Кто-то хочет сделать запрос на включение для поддержки URL-адреса rediss://?
Как насчет тебя @venkatesh-nadar 🙂 ?

@peterpoliwoda , кажется, уже добавлен в основную ветку... 👍

Хм ... да, я вижу, что это в репо, но npm не устанавливается с исправлением. Номер версии не изменился, поэтому «npm install» сохраняет старую сборку, как будто в архиве был старый архив.
Я заработал, установив прямо с Github с помощью
'''
установка npm https://github.com/NodeRedis/node_redis
'''

Огромное спасибо @venkatesh-nadar!

@BridgeAR Можем ли мы получить новый выпуск в npm, пожалуйста?

Несколько раз обращался к @BridgeAR с просьбой о доступе к публикации npm за последние годы и не слышал об этом, я знаю, что он занят вещами Node.js, и, вероятно, поэтому этим репозиторием пренебрегают.

Давайте обсудим будущее репо на https://github.com/NodeRedis/node_redis/issues/1040#issuecomment -581418899?

Всем привет, я стал ведущим сопровождающим и теперь имею все необходимые права доступа 🎉

Последние несколько дней я готовил master к выпуску, и несколько минут назад я только что опубликовал v3.0.0 в NPM; https://github.com/NodeRedis/node-redis/releases/tag/v3.0.0 , который включает это изменение.

Ожидайте регулярных релизов — мой главный приоритет на данный момент — сделать этот проект более удобным для участников, чтобы гарантировать, что проект живет и продолжает расти и не блокируется временем какого-либо отдельного человека. Для этого я хотел бы вырастить большее количество мелких участников. С этим я надеюсь смягчить предыдущую проблему проекта, нуждающегося в обновлениях, но нет никого, кто мог бы это сделать. Я работаю над этим;

  • [x] Предоставление документов и Кодекс поведения
  • [x] Настройка открытого коллектива и политики расходов участников

    • Вы заметите новую блестящую кнопку Sponsor в верхней части GitHub, я также пошел дальше и спонсировал ее сам и через свою компанию, чтобы помочь запустить ее для любых будущих участников.

  • WIP: автоматизация выпуска и семантическое управление версиями (публикация в NPM, создание журналов изменений и т. д.)
  • [x] Улучшить CI, например, Windows CI сейчас очень медленный и ненадежный

После этого я переключу свое внимание на модернизацию (например, промисы) и устранение технического долга в кодовой базе Node Redis. @BridgeAR уже сделал для этого кучу всего, если вам интересно, посмотрите ветку WIP v4 и ее журнал изменений.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги