Merge pull request #54 from fastify/issue-53

Fix issue 53

plugin.js

@@ -38,6 +38,7 @@ function fastifyCookieClearCookie (reply, name, options) {
 
 function onReqHandlerWrapper (options) {
   return function fastifyCookieOnReqHandler (fastifyReq, fastifyRes, done) {
+    fastifyReq.cookies = {} // New container per request. Issue #53
     const cookieHeader = fastifyReq.req.headers.cookie
     if (cookieHeader) {
       fastifyReq.cookies = cookie.parse(cookieHeader, options)

test/cookie.test.js

@@ -312,3 +312,32 @@ test('pass options to `cookies.parse`', (t) => {
     return str + 'test'
   }
 })
+
+test('issue 53', (t) => {
+  t.plan(5)
+  const fastify = Fastify()
+  fastify.register(plugin)
+
+  let cookies
+  let count = 1
+  fastify.get('/foo', (req, reply) => {
+    if (count > 1) {
+      t.notEqual(cookies, req.cookies)
+      return reply.send('done')
+    }
+
+    count += 1
+    cookies = req.cookies
+    reply.send('done')
+  })
+
+  fastify.inject({ url: '/foo' }, (err, response) => {
+    t.error(err)
+    t.is(response.body, 'done')
+  })
+
+  fastify.inject({ url: '/foo' }, (err, response) => {
+    t.error(err)
+    t.is(response.body, 'done')
+  })
+})