Yep that's perfect. Not being fixed about using require... it was just to
make code easy.


On Fri, May 3, 2013 at 11:15 AM, John Lyle <notifications@github.com> wrote:

> In lib/passportConfig.js:
>
> > +
> > +    /* No clever user handling here yet */
> > +    passport.serializeUser(function (user, done) {
> > +        done(null, user);
> > +    });
> > +
> > +    passport.deserializeUser(function (obj, done) {
> > +        done(null, obj);
> > +    });
> > +
> > +    return cb(passport, config);
> > +}
> > +
> > +function readAuthConfig(configName) {
> > +    var path = require('path');
> > +    var config = require(path.join('..', 'config.json'));
>
> I'm still having to use path.join, as this code is executed in the 'lib'
> directory. Is that OK, @habibvirji <https://github.com/habibvirji> ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/webinos/webinos-pzhWebServer/pull/1/files#r4071487>
> .
>