Allow init to accept an already configured cache client about express-stormpath HOT 5 CLOSED

I agree! The underlying Node SDK does support this, see the docs here and note the client option:

http://docs.stormpath.com/nodejs/api/client#redis

For a PR, I think we need to pass that option here:

https://github.com/stormpath/stormpath-express/blob/daaf31023043e5077b167566faf5740a0cb8345a/lib/stormpath.js#L39

from express-stormpath.

Thanks! Change seems straightforward enough, any suggestions for testing?

from express-stormpath.

Actually, it looks like there is already a request in progress for this same feature:
#56

Diff shows a change almost identical to what I made.

from express-stormpath.

Thanks everyone for commenting on this -- and sorry for my slow turnaround time.

I'm going to be merging the PR #56 into the library, and this is most definitely a needed / useful option.

I really appreciate ya'll bringing this to my attention <333

I'll leave another comment here once the PR has been merged, and a new release has been cut.

from express-stormpath.

Bam, new release has been cut! If you fetch 1.0.2 you'll be able to specify the cacheClient option when setting up the middleware =)

from express-stormpath.