Skip to content

Commit 4148e66

Browse files
committed
Fix Redis example to support Redis v4+ and Express 4/5
1 parent 256a3d1 commit 4148e66

File tree

1 file changed

+24
-11
lines changed

1 file changed

+24
-11
lines changed

examples/search/index.js

+24-11
Original file line numberDiff line numberDiff line change
@@ -19,28 +19,41 @@ var db = redis.createClient();
1919

2020
// npm install redis
2121

22+
// connect to Redis
23+
24+
db.connect()
25+
.catch((err) => console.error('Redis connection error:', err));
26+
2227
var app = express();
2328

2429
app.use(express.static(path.join(__dirname, 'public')));
2530

2631
// populate search
2732

28-
db.sadd('ferret', 'tobi');
29-
db.sadd('ferret', 'loki');
30-
db.sadd('ferret', 'jane');
31-
db.sadd('cat', 'manny');
32-
db.sadd('cat', 'luna');
33+
(async () => {
34+
try {
35+
await db.sAdd('ferret', 'tobi');
36+
await db.sAdd('ferret', 'loki');
37+
await db.sAdd('ferret', 'jane');
38+
await db.sAdd('cat', 'manny');
39+
await db.sAdd('cat', 'luna');
40+
} catch (err) {
41+
console.error('Error populating Redis:', err);
42+
}
43+
})();
3344

3445
/**
3546
* GET search for :query.
3647
*/
3748

38-
app.get('/search/:query?', function(req, res, next){
39-
var query = req.params.query;
40-
db.smembers(query, function(err, vals){
41-
if (err) return next(err);
42-
res.send(vals);
43-
});
49+
app.get('/search/:query{0,1}', function (req, res, next) {
50+
var query = req.params.query || '';
51+
db.sMembers(query)
52+
.then((vals) => res.send(vals))
53+
.catch((err) => {
54+
console.error(`Redis error for query "${query}":`, err);
55+
next(err);
56+
});
4457
});
4558

4659
/**

0 commit comments

Comments
 (0)