Use clone instead of Request::duplicate()

Toflar · Toflar · commit dcb3833b90b8 · 2018-06-19T15:06:03.000+02:00
diff --git a/src/Symfony/Component/HttpKernel/HttpCache/HttpCache.php b/src/Symfony/Component/HttpKernel/HttpCache/HttpCache.php
@@ -165,11 +165,11 @@ public function handle(Request $request, $type = HttpKernelInterface::MASTER_REQ
         // FIXME: catch exceptions and implement a 500 error page here? -> in Varnish, there is a built-in error page mechanism
         if (HttpKernelInterface::MASTER_REQUEST === $type) {
             $this->traces = array();
-            // Keep a duplicate of the original request for surrogates so they can access it.
-            // We must duplicate here to get a separate instance because the application will modify the request during
+            // Keep a clone of the original request for surrogates so they can access it.
+            // We must clone here to get a separate instance because the application will modify the request during
             // the application flow (we know it always does because we do ourselves by setting REMOTE_ADDR to 127.0.0.1
             // and adding the X-Forwarded-For header, see HttpCache::forward()).
-            $this->request = $request->duplicate();
+            $this->request = clone $request;
             if (null !== $this->surrogate) {
                 $this->surrogateCacheStrategy = $this->surrogate->createCacheStrategy();
             }
