Issue with 3rd party transports #285

merged 1 commit into from Aug 21, 2012


None yet
3 participants

polidore commented Aug 21, 2012

This fixes a bug with 3rd party transports. You were checking for an object, but it really should be a function in order to be able to do, for example:

ss.publish.transport.use(require('ss-mubsub'), {db: 'test'});

And have the config passed properly into the exports instantiation function.

ss-mubsub won't work until this is fixed.

This pull request passes (merged eea863e into 48a98ea).

owenb commented Aug 21, 2012

Thank you!

owenb merged commit 2db0ff4 into socketstream:master Aug 21, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment