Node-redis: त्रुटि, जो retry_strategy से वापस आती है, ध्यान में नहीं रहती है

@pavelsc मैंने इसे पुन: पेश करने की कोशिश की लेकिन अभी तक मैं नहीं कर सका।

कृपया किसी तीसरे पक्ष के मॉड्यूल के बिना इस मुद्दे को पुन: पेश करने का प्रयास करें। वर्तमान में आप कम से कम q उपयोग करते प्रतीत होते हैं।

मुझे एक ही त्रुटि का सामना करना पड़ रहा है। अगर मैं जानबूझकर रेडिस क्लाइंट को एक खराब यूआरएल प्रदान करता हूं, तो on.error विधि लागू नहीं होती है। ये रहा एक सरल उदाहरण:

var redis = require("redis");

exports.handler = function (event, context, callback) {

    console.log("Executing test lambda for diagnosing redis issues");

    var redisInfo = {
        HOST: process.env.REDIS_HOST,
        PORT: process.env.REDIS_PORT
    };

    console.log(process.env.REDIS_HOST);
    console.log(process.env.REDIS_PORT);

    console.log("Connecting to Redis...");

    var client = redis.createClient({
        host: redisInfo.HOST,
        port: redisInfo.PORT,
        retry_strategy: function (options) {

            if (options.total_retry_time > 2000) {
                console.log("throwing an error...");
                return new Error('Retry time exhausted');
            }

            return 200;
        }
    });

    // if you'd like to select database 3, instead of 0 (default), call
    // client.select(3, function() { /* ... */ });

    client.on("error", function (err) {
        console.log("Error " + err);
        callback(null, "Error with Redis");
    });

    client.on('connect', function() {
        console.log("Connected to Redis");
    });

    client.on('end', function() {
        console.log("Redis end");
    });

    client.set("string key", "string val", redis.print);
    client.hset("hash key", "hashtest 1", "some value", redis.print);
    client.hset(["hash key", "hashtest 2", "some other value"], redis.print);
    client.hkeys("hash key", function (err, replies) {
        console.log(replies.length + " replies:");
        replies.forEach(function (reply, i) {
            console.log("    " + i + ": " + reply);
        });
        client.quit();
    });

    client.quit();

    callback(null, "Success");
};

अभी के लिए मैं connect_timeout का उपयोग करने के लिए वापस लौट रहा हूं, जो कनेक्ट टाइमआउट समाप्त होने के बाद सही ढंग से 'त्रुटि' पर निकलता है।

मेरे पास एक ही समस्या है, एक खराब एंडपॉइंट के साथ एक कस्टम retry_strategy का उपयोग करके "AbortError:" में समाप्त होता है

इसने मुझे आज भी पकड़ लिया। कोड को संक्षेप में देखने से, यह जानबूझकर किया गया व्यवहार प्रतीत होता है। https://github.com/NodeRedis/node_redis/blob/79558c524ff783000a6027fb159739770f98b10e/index.js#L405 स्पष्ट रूप से बताता है कि यदि retry_strategy सेट किया गया है, तो त्रुटि का उत्सर्जन न करें और इसके बजाय इसे फेंकना जारी रखें। मुझे यह जानने की उत्सुकता होगी कि ऐसा क्यों है, हालांकि ऐसा कोई कारण नहीं लगता है कि यह एक त्वरित नज़र से फेंकने के बजाय इसे उत्सर्जित नहीं कर सकता है। क्या कोई कारण है कि यह सशर्त नहीं निकाला जा सका, ताकि त्रुटि हमेशा उत्सर्जित हो?

मुझे भी यह समस्या लगातार हो रही है।

मैं भी ENOTFOUND प्राप्त करते समय त्रुटियों को पकड़ने में सक्षम नहीं हूं।

{
    host: "foo",
    retry_strategy: function (options) {
        if (options.error && options.error.code === "ENOTFOUND") {
            return new Error("The server was not found");
        }

        // reconnect after
        return 1000;
}

साथ:

redis.on("error", err => {
    console.error("Cache Error: " + err);
});

एप्लिकेशन को डीबग करना, मैं ENOTFOUND चेक में शामिल हो रहा हूं जैसा कि retry_strategy में ऊपर बताया गया है लेकिन यह त्रुटि ईवेंट हैंडलर का आह्वान नहीं कर रहा है।

मेरे पास एक ही समस्या है, स्रोत कोड के माध्यम से खुदाई करने के बाद मैंने पाया कि अगर हम बदलते हैं
यह लाइन (या डीबग मोड सक्षम करें)
https://github.com/NodeRedis/node_redis/blob/009479537eb920d2c34045026a55d31febd1edd7/index.js#L381 -L382

और कोड का यह टुकड़ा यहां डालें (सरणी में तुरंत त्रुटि जोड़ें)
https://github.com/NodeRedis/node_redis/blob/009479537eb920d2c34045026a55d31febd1edd7/index.js#L352 -L353

    if (options.error) {
      aggregated_errors.push(options.error);
    }

यह काम करता है और 'त्रुटि' को सही ढंग से उत्सर्जित करता है।

उस फ़ंक्शन में नेस्टेड लूप निष्पादित नहीं होता है, क्योंकि 'command_queue' खाली है और त्रुटि कभी भी सरणी में नहीं जुड़ती है और इस प्रकार उत्सर्जित नहीं होती है। अगर मैं सही ढंग से समझूं , तो यह काफी पुराना कोड है, इसलिए हमें अनुरक्षकों से या

मैंने यह भी देखा कि पहले असफल कनेक्शन पर 'अंत' घटना का उत्सर्जन होता है, इसका मतलब कुछ (या नहीं) हो सकता है, मैंने दो दिन पहले रेडिस को उठाया था, इसलिए यह सुनिश्चित नहीं है कि आंतरिक अभी तक कैसे काम करते हैं। जब मेरे पास समय होगा तो मैं थोड़ा और कांटा और खुदाई करने की कोशिश करूंगा।

और सचमुच अगला अंक इस समस्या से जुड़ा हुआ लगता है #1198

@ v1adko मैं वर्तमान में यात्रा कर रहा हूं, लेकिन मैं इसे आज या कल बाद में देखने की कोशिश करूंगा (जब तक कि रूबेन मुझे इसमें नहीं मारता)।

त्रुटि परिदृश्यों का परीक्षण करने के लिए मेरे पास मेरा रेडिस यूआरएल जानबूझकर गलत है, लेकिन मुझे लगता है कि रेडिस से कनेक्ट करने का प्रयास करते समय मेरी retry_strategy को लागू नहीं किया गया है। कनेक्शन बंद होने पर ही retry_strategy लागू किया जाता है।

`कॉन्स्ट रेडिस = आवश्यकता ('रेडिस');
कॉन्स्ट लॉग = आवश्यकता ('./logUtil')। लकड़हारा;

मॉड्यूल.एक्सपोर्ट्स.कनेक्ट = () => {

var redisRetryStrategy = function(options) {
    if (options.error && options.error.code === 'ECONNREFUSED') {
        // End reconnecting on a specific error and flush all commands with 
        // a individual error
        log.error('The redis server refused the connection');
        return new Error('The redis server refused the connection');
    }

    log.info(`Already spent ${options.total_retry_time} milliseconds to re-establish connection with redis`);
    if (options.total_retry_time > 2000) {
        // End reconnecting after a specific timeout and flush all commands 
        // with a individual error 
        log.error('Retry time exhausted');
        return new Error('Retry time exhausted');
    }
    log.info(`Attempting ${options.attempt} time to establish connection with redis`);
    if (options.attempt > 5) {
        // End reconnecting with built in error 
        log.error('Exhausted the retry attempts to establish connection to redis');
        return undefined;
    }
    // reconnect after 
    return 100;
}


log.info(`Redis connection url is :${process.env.REDIS_URL}`);
var redisClient = redis.createClient(qualifyUrl(process.env.REDIS_URL), {
    retry_strategy: redisRetryStrategy
});

redisClient.offline_queue_length = 3;

redisClient.on('connect', function() {
    console.log('Connected to Redis');
});

redisClient.on('reconnecting', function() {
    console.log('Re-Connecting to Redis');
});

redisClient.on('error', (err)=> {
    console.log(`Error trying to create redis connection: ${JSON.stringify(err)}`);
});
return redisClient;

}

कॉन्स क्वालिफाईयूआरएल = (यूआरएल) => {
वापसी '//' + url.replace(/^\/+/,"");
};

`

क्या कोई इस मुद्दे को हल करने में मेरी मदद कर सकता है।

मेरा भी यही विचार है। ऐसा लगता है कि यह बुरा हैक अपेक्षित व्यवहार बनाता है, लेकिन यह सुनिश्चित नहीं है कि इसका कोई व्यापक प्रभाव है या नहीं:

const client = redis.createClient({
  retry_strategy: ({error}) => client.emit('error', error)
});

client.on('error', console.error);

मुझे इस समय वही समस्याएं आ रही हैं। Retry_strategy का उपयोग करके, रीडमी में उदाहरण के अनुसार त्रुटि को वापस करना, फिर भी क्लाइंट द्वारा कोई त्रुटि उत्सर्जित नहीं की जाती है। @ v1adko द्वारा प्रस्तावित सुधार कम से कम अंकित मूल्य पर समस्या का समाधान करते हैं।

मैं सोच रहा हूँ कि यहाँ उल्लिखित बैकवर्ड असंगति क्या है?
https://github.com/NodeRedis/node_redis/blob/009479537eb920d2c34045026a55d31febd1edd7/index.js#L380

जैसा कि @maael द्वारा इंगित किया गया है, व्यवहार जानबूझकर प्रतीत होता है जब retry_strategy सेट किया जाता है। तो क्या व्यवहार अपेक्षित है, लेकिन दस्तावेज़ीकरण गलत है? क्या मुझे @c24w द्वारा सुझाए गए अनुसार मैन्युअल रूप से क्लाइंट के लिए त्रुटियों का उत्सर्जन करना चाहिए?

संपादित करें: जैसा कि मैं पैकेज में खुदाई कर रहा हूं, मुझे एहसास हो रहा है कि मैन्युअल रूप से उत्सर्जन करना शायद आगे का रास्ता नहीं है। ऐसा लगता है कि मुझे उल्लिखित ब्रेकिंग परिवर्तनों को समझने की जरूरत है।

कोई खबर ? मेरी भी यही समस्या है।

कोई खबर?

करना एक गलत विचार है:
js if (options.error && options.error.code === 'ECONNREFUSED') { // End reconnecting on a specific error and flush all commands with // a individual error return Math.min(options.attempt * 100, 3000); }

एक ही समस्या है, retry_Strategy त्रुटि घटना को ट्रिगर नहीं कर रहा है, अभी तक कोई फिक्स नहीं है?

क्या कोई सफल हुआ?

हमने इसके बजाय अपने कार्यान्वयन को https://github.com/luin/ioredis पर स्विच कर दिया, जिससे कुछ सुधार हुए (मूल वादे, आलसी कनेक्ट (रेडिस क्लाइंट को इंस्टाल करते समय कनेक्शन खोलने से बचें, हमें त्रुटियों को ठीक उसी तरह से संभालने में मदद करता है जहां हमें चाहिए)), और निम्नलिखित कोड को चलाने की अनुमति देता है:

let cachedItem;

  try {
    logger.debug(`Fetching GraphCMS query in redis cache...`);
    // XXX If fetching data from redis fails, we will fall back to running the query against GraphCMS API in order to ensure the client gets the data anyway
    cachedItem = await redisClient.get(body);
  } catch (e) {
    logger.debug(`An exception occurred while fetching redis cache.`);
    logger.error(e);
    epsagon.setError(e);
  }

निम्नलिखित utils/redis.js का उपयोग करना:

import { createLogger } from '@unly/utils-simple-logger';
import Redis from 'ioredis';
import epsagon from './epsagon';

const logger = createLogger({
  label: 'Redis client',
});

/**
 * Creates a redis client
 *
 * <strong i="11">@param</strong> url Url of the redis client, must contain the port number and be of the form "localhost:6379"
 * <strong i="12">@param</strong> password Password of the redis client
 * <strong i="13">@param</strong> maxRetriesPerRequest By default, all pending commands will be flushed with an error every 20 retry attempts.
 *          That makes sure commands won't wait forever when the connection is down.
 *          Set to null to disable this behavior, and every command will wait forever until the connection is alive again.
 * <strong i="14">@return</strong> {Redis}
 */
export const getClient = (url = process.env.REDIS_URL, password = process.env.REDIS_PASSWORD, maxRetriesPerRequest = 20) => {
  const client = new Redis(`redis://${url}`, {
    password,
    showFriendlyErrorStack: true, // See https://github.com/luin/ioredis#error-handling
    lazyConnect: true, // XXX Don't attempt to connect when initializing the client, in order to properly handle connection failure on a use-case basis
    maxRetriesPerRequest,
  });

  client.on('connect', function () {
    logger.info('Connected to redis instance');
  });

  client.on('ready', function () {
    logger.info('Redis instance is ready (data loaded from disk)');
  });

  // Handles redis connection temporarily going down without app crashing
  // If an error is handled here, then redis will attempt to retry the request based on maxRetriesPerRequest
  client.on('error', function (e) {
    logger.error(`Error connecting to redis: "${e}"`);
    epsagon.setError(e);
  });

  return client;
};

और utils/redis.test.js फ़ाइल:

import { getClient } from './redis';

let redisClient;
let redisClientFailure;

describe('utils/redis.js', () => {
  beforeAll(() => {
    redisClient = getClient();
    redisClientFailure = getClient('localhost:5555', null, 0); // XXX This shouldn't throw an error because we're using lazyConnect:true which doesn't automatically connect to redis
  });

  afterAll(async () => {
    await redisClient.quit();
    await redisClientFailure.quit();
  });

  describe('should successfully init the redis client', () => {
    test('when provided connection info are correct', async () => {
      // Environment variables are from the .env.test file - This tests a localhost connection only
      expect(redisClient.options.host).toEqual(process.env.REDIS_URL.split(':')[0]);
      expect(redisClient.options.port).toEqual(parseInt(process.env.REDIS_URL.split(':')[1], 10));
      expect(redisClient.options.password).toEqual(process.env.REDIS_PASSWORD);
    });

    test('when connection info are incorrect', async () => {
      expect(redisClientFailure.options.host).toEqual('localhost');
      expect(redisClientFailure.options.port).toEqual(5555);
    });
  });

  describe('should successfully perform native operations (read/write/delete/update)', () => {
    test('when using async/await (using native node.js promises)', async () => {
      const setResult = await redisClient.set('key-1', 'value-1');
      expect(setResult).toEqual('OK');

      const result = await redisClient.get('key-1');
      expect(result).toEqual('value-1');

      const delResult = await redisClient.del('key-1');
      expect(delResult).toEqual(1);

      const setResultB = await redisClient.set('key-1', 'value-1b');
      expect(setResultB).toEqual('OK');

      const resultB = await redisClient.get('key-1');
      expect(resultB).toEqual('value-1b');

      const setResultC = await redisClient.set('key-1', 'value-1c');
      expect(setResultC).toEqual('OK');

      const resultC = await redisClient.get('key-1');
      expect(resultC).toEqual('value-1c');
    });
  });

  describe('should allow to catch an error when failing to open a connection to redis, in order to gracefully handle the error instead of crashing the app', () => {
    test('when connection info are incorrect', async () => {
      expect(redisClientFailure.options.host).toEqual('localhost');
      expect(redisClientFailure.options.port).toEqual(5555);

      try {
        await redisClientFailure.set('key-1', 'value-1'); // This should throw an error, because the connection to redis will be made when executing the
        expect(true).toBe(false); // This shouldn't be called, or the test will fail
      } catch (e) {
        expect(e).toBeDefined();
        expect(e.message).toContain('Reached the max retries per request limit');
      }
      await redisClientFailure.quit();
    });
  });
});

एनवी चर:

REDIS_URL=localhost:6379
REDIS_PASSWORD=mypasswordissostrong