node.js connect-redis redis.get(key,callback) return null

Question

in connect-redis to return session value use .get(sid, callback) but it always return null so I think its not save in redis but I check it session save in redis correctly after investigation find problem is with Sid see /connect-redis/lib/connect-redis.js

  RedisStore.prototype.get = function(sid, fn){
   sid =this.prefix + sid;
    debug('GET "%s"', sid);

    console.log('lib');
    console.log(sid);

    this.client.get( sid , function(err, data){

      if (err) return fn(err);
      if (!data) return fn(null,'');
      var result;
      data = data.toString();
      debug('GOT %s', data);
      try {
        result = JSON.parse(data); 
      } catch (err) {
        return fn(err);
      }

      return fn(null, result);
    });
  };

if I but sessionID as string it will return session

  RedisStore.prototype.get = function(sid, fn){

  //  sid =this.prefix + sid;
    debug('GET "%s"', sid);

    console.log('lib');
    console.log(sid);

    this.client.get( 'lweUw//EdbygbGr/2gAZt0kb' , function(err, data){

      if (err) return fn(err);
      if (!data) return fn(null,'');
      var result;
      data = data.toString();
      debug('GOT %s', data);
      try {
        result = JSON.parse(data); 
      } catch (err) {
        return fn(err);
      }

      return fn(null, result);
    });
  };

Note:

typeof(sid) is string even if add qoute to sid like this "\'sid\'" it will return null

It would be helpful to see your callback function.

Eric Elliott
– Eric Elliott

2013-08-05 09:22:00 +00:00
Commented Aug 5, 2013 at 9:22 — Eric Elliott
– Eric Elliott, Commented Aug 5, 2013 at 9:22

Eric Elliott · Accepted Answer · 2013-08-04 09:19:54Z

2

The callback takes the form callback(err, data). If there is no error, the first parameter will always be null. The real response is in the second parameter.

See: return fn(null, result); in .get().

answered Aug 4, 2013 at 9:19

Eric Elliott

4,7611 gold badge29 silver badges33 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

april Over a year ago

no data return the response it return from if condition see (!data) return fn(null,'');

Eric Elliott Over a year ago

You said it returned null in your question. An empty string is not null. Is the type of the return value a string?

Collectives™ on Stack Overflow

node.js connect-redis redis.get(key,callback) return null

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related