Value from client.get() is "true" instead of the real value

Question

I am using nowjs and node_redis. I am trying to create something very simple. But so far, the tutorial have left me blank because they only do console.log().

//REDIS
var redis = require("redis"),
    client = redis.createClient();

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

client.set("card", "apple");

everyone.now.signalShowRedisCard = function() {
    nowjs.getGroup(this.now.room).now.receiveShowRedisCard(client.get("card").toString());
}

In my client side:

now.receiveShowRedisCard = function(card_id) {
    alert("redis card: "+card_id);
}

The alert only gives out "true" - I was expecting to get the value of the key "card" which is "apple".

Any ideas?

score 18 · Accepted Answer · answered Aug 03 '11 at 10:17

You are trying to use an async library in a sync way. This is the right way:

//REDIS
var redis = require("redis"),
    client = redis.createClient();

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

client.set("card", "apple", function(err) {
    if (err) throw err;
});

everyone.now.signalShowRedisCard = function() {
    var self = this;
    client.get("card", function (err, res) {
        nowjs.getGroup(self.now.room).now.receiveShowRedisCard(res);
    });
}

score 8 · Answer 2 · answered Feb 23 '19 at 18:49

8

One option is to use Bluebird to turn Redis callbacks into promises. Then you can use it with .then() or async/await.

import redis from 'redis'
import bluebird from 'bluebird'

bluebird.promisifyAll(redis)
const client = redis.createClient()

await client.set("myKey", "my value")
const value = await client.getAsync("myKey")

Notice your methods should have Async appened to them.

answered Feb 23 '19 at 18:49

crash springfield

1,072
7
17
33

Can you export the client to another module for use? I am trying to export it and it still returns `true` for queries. – Mark A Jul 06 '19 at 20:24

bereket gebredingle · Answer 3 · 2019-05-12T06:19:17.713

4

Use Async Redis

npm i async-redis --save

const asyncRedis = require("async-redis");    
const client = asyncRedis.createClient(); 

await client.set("string key", "string val");
const value = await client.get("string key");

console.log(value);

await client.flushall("string key");

edited May 12 '19 at 06:19

answered Dec 31 '18 at 14:23

bereket gebredingle

12,064
3
36
47

score 1 · Answer 4 · answered Jan 24 '22 at 10:18

1

You can also use a function provided by node_redis library
const getAsync = promisify(client.get).bind(client);
and use this to get values from redis as follows
const value = await getAsync(key)

answered Jan 24 '22 at 10:18

Tejas

61
4

Value from client.get() is "true" instead of the real value

4 Answers4

Use Async Redis