Skip to content

Commit

Permalink
make redis errors clearer
Browse files Browse the repository at this point in the history
  • Loading branch information
jackkav committed Jan 8, 2024
1 parent 48072b0 commit 0118766
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 14 deletions.
25 changes: 12 additions & 13 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,21 +26,8 @@ Mockbin is used internally and maintained by [Kong](https://github.com/Kong), wh

## Installation

```shell
git clone https://github.com/Kong/mockbin.git ./mockbin
cd mockbin
cp .env.sample .env
brew install fnm
fnm use
npm install
```

Note: nvm, n or volta can be used instead of fnm.

### Requirements

other than the dependencies listed in [package.json](package.json) The following are required:

- [Redis](http://redis.io/)

```shell
Expand All @@ -49,6 +36,18 @@ brew services start redis
```

Redis should be now running on localhost:6379
Mockbin will start without redis but you wont be able to set or get response bins.

```shell
git clone https://github.com/Kong/mockbin.git ./mockbin
cd mockbin
cp .env.sample .env
brew install fnm
fnm use
npm install
```

Note: nvm, n or volta can be used instead of fnm.

### Running with Node

Expand Down
2 changes: 1 addition & 1 deletion lib/routes/bins.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ module.exports = function bins(dsnStr) {
}

this.client.on("error", (err) => {
debug("redis error:", err);
console.log("redis error:", err);
});

const router = express.Router();
Expand Down

0 comments on commit 0118766

Please sign in to comment.