Skip to content

Commit

Permalink
fix: 🐛 Removed unused redis dependency (#35)
Browse files Browse the repository at this point in the history
  • Loading branch information
JohanObrink authored Sep 19, 2019
1 parent 796d5fa commit b707454
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 205 deletions.
39 changes: 0 additions & 39 deletions lib/adapters/redis.js

This file was deleted.

11 changes: 1 addition & 10 deletions lib/routes/health.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,13 @@
const { Router } = require('express')

const postgres = require('../adapters/postgres')
const redis = require('../adapters/redis')

const router = Router()

router.get('/', async (req, res, next) => {
let statusCode = 200
const status = {
postgres: '?',
redis: '?'
postgres: '?'
}

try {
Expand All @@ -20,13 +18,6 @@ router.get('/', async (req, res, next) => {
status.postgres = '!OK'
}

if (redis.status() === 'ready') {
status.redis = 'OK'
} else {
statusCode = 503
status.redis = '!OK'
}

res.status(statusCode).send(status)
})

Expand Down
155 changes: 0 additions & 155 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
"elastic-apm-node": "2.11.1",
"express": "4.16.4",
"http-errors": "1.7.2",
"ioredis": "3.2.2",
"memory-fs": "0.4.1",
"merge": "1.2.1",
"moment": "2.24.0",
Expand Down

0 comments on commit b707454

Please sign in to comment.