Node.js & Redis Caching

node_redis_cache.js

const express = require('express');
const fetch = require('node-fetch');
const redis = require('redis');

const PORT = process.env.PORT || 5000;
const REDIS_PORT = process.env.PORT || 6379;

const client = redis.createClient(REDIS_PORT);

const app = express();

// Set response
function setResponse(username, repos) {
  return `<h2>${username} has ${repos} Github repos</h2>`;
}

// Make request to Github for data
async function getRepos(req, res, next) {
  try {
    console.log('Fetching Data...');

    const { username } = req.params;

    const response = await fetch(`https://api.github.com/users/${username}`);

    const data = await response.json();

    const repos = data.public_repos;

    // Set data to Redis
    client.setex(username, 3600, repos);

    res.send(setResponse(username, repos));
  } catch (err) {
    console.error(err);
    res.status(500);
  }
}

// Cache middleware
function cache(req, res, next) {
  const { username } = req.params;

  client.get(username, (err, data) => {
    if (err) throw err;

    if (data !== null) {
      res.send(setResponse(username, data));
    } else {
      next();
    }
  });
}

app.get('/repos/:username', cache, getRepos);

app.listen(5000, () => {
  console.log(`App listening on port ${PORT}`);
});

Resolved: Keep await client.connect()

Put data in a variable and await the response. See function:

async function cache(req, res, next) {
  const { username } = req.params;
  const data = await client.get(username);
  if (data !== null) {
    res.send(setResponse(username, data));
  } else {
    next();
  }
}

I solved the problem : "ClientClosedError: The client is closed"

1. Stating redis in the terminal:

   @pop-os:~$ redis-server

2. adding in the code: client.connect();

   that is:

   const client = redis.createClient(REDIS_PORT)
   client.connect();
   client.on('connect', () => {
   console.log('connected');
   });

@MoAboabdo

This code did not work for me idk why! this error comes out
ClientClosedError: The client is closed

try to downgrade your redis and node-fetch packages. It will solve your issue, I have the same issue, now works fine.

my case const fetch = require('node-fetch') is not working

my case const fetch = require('node-fetch') is not working

Instal the used version so you don't have to worry about importing it in a different way.
npm i [email protected]