derbyjs / derby-starter Goto Github PK

No compatible version found: derby@^0.6.0
True, derby@^0.6.0 not in valid install targets list.

There are a lots of unnecessary packages in package.json. Could you clear it. I have a problem with hiredis installarion in my Windows system. But it is not even required. Please remove it or move to optionalDependencies block (it's a good practic).

Thank you!

Derby store is correctly configured with redis settings with

var redisClient;
  if (process.env.REDIS_HOST) {
    redisClient = redis.createClient(process.env.REDIS_PORT, process.env.REDIS_HOST);
    redisClient.auth(process.env.REDIS_PASSWORD);
  } else if (process.env.OPENREDIS_URL) {
    var redisUrl = parseUrl(process.env.OPENREDIS_URL);
    redisClient = redis.createClient(redisUrl.port, redisUrl.hostname);
    redisClient.auth(redisUrl.auth.split(":")[1]);
  } else {
    redisClient = redis.createClient();
  }

Yet session store uses different code

store: new RedisStore({
        host: process.env.REDIS_HOST || "localhost",
        port: process.env.REDIS_PORT || 6379
    })

hence app fails to start when using non-default settings for redis.

Right now redis config var can be read from OpenRedis (OPENREDIS_URL) config, yet it does not have free tier at the moment.

Please add possibility to read redis config var for RedisCloud (REDISCLOUD_URL). It has free tier and can be used for Heroku deployments of the sample apps.

Some of the dependencies in package.json are old. The specific issue I ran into is that hiredis resulted in build errors when using Node.js 4.1.2. The build works find with a newer version of hiredis.

Does it makes sense to update the dependencies and then submit a PR?