cookies(req, res, [options])

[jonathanong]

i think setting the options every time you .get() or .set() the cookies is pretty annoying. we should able to use default options passed to the constructor.

[dougwilson]

Makes sense.

[jonathanong]

going to change this to:

var cookies = new Cookies([options])

app.use(function (req, res, next) {
  req.cookies = res.cookies = cookies(req, res)
  next()
})

[shash7]

This still isn't implement eh?

[jonathanong]

#48

a code review would be nice

[shash7]

I just checked the source code and it should be like :

var opts = {};
function Cookies(req, res, opts) {
    // Check if only opts is given
    if(arguments.length === 1) {
        opts = req;
        return false; // Don't know about this part
    } else {
    // Continue as usual
    }
}