don't init req.cookies before checking the header (#248)

plugin.js

@@ -9,7 +9,7 @@ const kReplySetCookies = Symbol('fastify.reply.setCookies')
 const kReplySetCookiesHookRan = Symbol('fastify.reply.setCookiesHookRan')
 
 function fastifyCookieSetCookie (reply, name, value, options) {
-  parseCookies(reply.context.server, reply.request, reply)
+  parseCookies(reply.server, reply.request, reply)
 
   const opts = Object.assign({}, options)
 
@@ -51,11 +51,9 @@ function fastifyCookieClearCookie (reply, name, options) {
 function parseCookies (fastify, request, reply) {
   if (reply[kReplySetCookies]) return
 
-  request.cookies = {} // New container per request. Issue #53
   const cookieHeader = request.raw.headers.cookie
-  if (cookieHeader) {
-    request.cookies = fastify.parseCookie(cookieHeader)
-  }
+
+  request.cookies = cookieHeader ? fastify.parseCookie(cookieHeader) : {} // New container per request. Issue #53
   reply[kReplySetCookies] = new Map()
 }
 
@@ -116,18 +114,6 @@ function fastifyCookieOnSendHandler (fastifyReq, fastifyRes, payload, done) {
   done()
 }
 
-function getHook (hook = 'onRequest') {
-  const hooks = {
-    onRequest: 'onRequest',
-    preParsing: 'preParsing',
-    preValidation: 'preValidation',
-    preHandler: 'preHandler',
-    [false]: false
-  }
-
-  return hooks[hook]
-}
-
 function plugin (fastify, options, next) {
   const secret = options.secret
   const hook = getHook(options.hook)
@@ -190,6 +176,18 @@ function plugin (fastify, options, next) {
   }
 }
 
+function getHook (hook = 'onRequest') {
+  const hooks = {
+    onRequest: 'onRequest',
+    preParsing: 'preParsing',
+    preValidation: 'preValidation',
+    preHandler: 'preHandler',
+    [false]: false
+  }
+
+  return hooks[hook]
+}
+
 function isConnectionSecure (request) {
   return (
     request.raw.socket?.encrypted === true ||