Skip to content

Commit 1311183

Browse files
cparticacpartica
committed
Issue-230: adding varnish
- fixing dispatch
1 parent a9299e1 commit 1311183

File tree

16 files changed

+47
-78
lines changed

16 files changed

+47
-78
lines changed

‎app/code/Magento/CatalogGraphQl/Model/Resolver/Category/CategoriesIdentity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class CategoriesIdentity implements IdentityInterface
1818
* Get category IDs from resolved data
1919
*
2020
* @param array $resolvedData
21-
* @return array
21+
* @return string[]
2222
*/
2323
public function getIdentities(array $resolvedData): array
2424
{

‎app/code/Magento/CatalogGraphQl/Model/Resolver/Category/CategoryTreeIdentity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class CategoryTreeIdentity implements IdentityInterface
1818
* Get category ID from resolved data
1919
*
2020
* @param array $resolvedData
21-
* @return array
21+
* @return string[]
2222
*/
2323
public function getIdentities(array $resolvedData): array
2424
{

‎app/code/Magento/CatalogGraphQl/Model/Resolver/Product/Identity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class Identity implements IdentityInterface
1818
* Get product ids for cache tag
1919
*
2020
* @param array $resolvedData
21-
* @return array
21+
* @return string[]
2222
*/
2323
public function getIdentities(array $resolvedData): array
2424
{

‎app/code/Magento/CmsGraphQl/Model/Resolver/Block/Identity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class Identity implements IdentityInterface
1919
* Get block identities from resolved data
2020
*
2121
* @param array $resolvedData
22-
* @return array
22+
* @return string[]
2323
*/
2424
public function getIdentities(array $resolvedData): array
2525
{

‎app/code/Magento/CmsGraphQl/Model/Resolver/Page/Identity.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class Identity implements IdentityInterface
1919
* Get page ID from resolved data
2020
*
2121
* @param array $resolvedData
22-
* @return array
22+
* @return string[]
2323
*/
2424
public function getIdentities(array $resolvedData): array
2525
{

‎app/code/Magento/GraphQl/Controller/GraphQl.php

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515
use Magento\Framework\GraphQl\Query\Resolver\ContextInterface;
1616
use Magento\Framework\GraphQl\Schema\SchemaGeneratorInterface;
1717
use Magento\Framework\Serialize\SerializerInterface;
18-
use Magento\Framework\Controller\ResultInterface;
1918
use Magento\Framework\Webapi\Response;
19+
use Magento\Framework\App\Response\Http as HttpResponse;
2020
use Magento\Framework\GraphQl\Query\Fields as QueryFields;
2121
use Magento\Framework\Controller\Result\JsonFactory;
2222
use Magento\Framework\App\ObjectManager;
@@ -30,7 +30,7 @@
3030
class GraphQl implements FrontControllerInterface
3131
{
3232
/**
33-
* @var Response
33+
* @var \Magento\Framework\Webapi\Response
3434
* @deprecated
3535
*/
3636
private $response;
@@ -75,6 +75,11 @@ class GraphQl implements FrontControllerInterface
7575
*/
7676
private $jsonFactory;
7777

78+
/**
79+
* @var HttpResponse
80+
*/
81+
private $httpResponse;
82+
7883
/**
7984
* @param Response $response
8085
* @param SchemaGeneratorInterface $schemaGenerator
@@ -85,6 +90,8 @@ class GraphQl implements FrontControllerInterface
8590
* @param HttpRequestProcessor $requestProcessor
8691
* @param QueryFields $queryFields
8792
* @param JsonFactory|null $jsonFactory
93+
* @param HttpResponse|null $httpResponse
94+
* @SuppressWarnings(PHPMD.ExcessiveParameterList)
8895
*/
8996
public function __construct(
9097
Response $response,
@@ -95,7 +102,8 @@ public function __construct(
95102
ContextInterface $resolverContext,
96103
HttpRequestProcessor $requestProcessor,
97104
QueryFields $queryFields,
98-
JsonFactory $jsonFactory = null
105+
JsonFactory $jsonFactory = null,
106+
HttpResponse $httpResponse = null
99107
) {
100108
$this->response = $response;
101109
$this->schemaGenerator = $schemaGenerator;
@@ -106,15 +114,16 @@ public function __construct(
106114
$this->requestProcessor = $requestProcessor;
107115
$this->queryFields = $queryFields;
108116
$this->jsonFactory = $jsonFactory ?:ObjectManager::getInstance()->get(JsonFactory::class);
117+
$this->httpResponse = $httpResponse ?:ObjectManager::getInstance()->get(HttpResponse::class);
109118
}
110119

111120
/**
112121
* Handle GraphQL request
113122
*
114123
* @param RequestInterface $request
115-
* @return ResultInterface
124+
* @return Response
116125
*/
117-
public function dispatch(RequestInterface $request)
126+
public function dispatch(RequestInterface $request) : HttpResponse
118127
{
119128
$statusCode = 200;
120129
$jsonResult = $this->jsonFactory->create();
@@ -145,7 +154,8 @@ public function dispatch(RequestInterface $request)
145154

146155
$jsonResult->setHttpResponseCode($statusCode);
147156
$jsonResult->setData($result);
148-
return $jsonResult;
157+
$jsonResult->renderResult($this->httpResponse);
158+
return $this->httpResponse;
149159
}
150160

151161
/**

‎app/code/Magento/GraphQlCache/Controller/Plugin/GraphQl.php

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,15 @@
99

1010
use Magento\Framework\App\FrontControllerInterface;
1111
use Magento\Framework\App\RequestInterface;
12-
use Magento\Framework\App\ResponseInterface;
1312
use Magento\GraphQlCache\Model\CacheableQuery;
1413
use Magento\Framework\App\Response\Http as HttpResponse;
1514
use Magento\Framework\Controller\ResultInterface;
1615
use Magento\PageCache\Model\Config;
1716
use Magento\GraphQl\Controller\HttpRequestProcessor;
17+
use Magento\Framework\App\Response\Http as ResponseHttp;
1818

1919
/**
20-
* Class Plugin
20+
* Plugin for handling controller after controller tags and pre-controller validation.
2121
*/
2222
class GraphQl
2323
{
@@ -76,24 +76,19 @@ public function beforeDispatch(
7676
}
7777

7878
/**
79-
* Plugin for GraphQL after dispatch to set tag and cache headers
79+
* Plugin for GraphQL after render from dispatch to set tag and cache headers
8080
*
81-
* The $response doesn't have a set type because it's alternating between ResponseInterface and ResultInterface
82-
* depending if it comes from builtin cache or the dispatch.
81+
* @param ResultInterface $subject
82+
* @param ResultInterface $result
83+
* @param ResponseHttp $response
84+
* @return ResultInterface
8385
*
84-
* @param FrontControllerInterface $subject
85-
* @param ResponseInterface|ResultInterface $response
86-
* @param RequestInterface $request
87-
* @return ResponseInterface|ResultInterface
8886
* @SuppressWarnings(PHPMD.UnusedFormalParameter)
8987
*/
90-
public function afterDispatch(
91-
FrontControllerInterface $subject,
92-
$response,
93-
RequestInterface $request
94-
) {
88+
public function afterRenderResult(ResultInterface $subject, ResultInterface $result, ResponseHttp $response)
89+
{
9590
$sendNoCacheHeaders = false;
96-
if ($this->config->isEnabled() && $request->isGet()) {
91+
if ($this->config->isEnabled()) {
9792
if ($this->cacheableQuery->shouldPopulateCacheHeadersWithTags()) {
9893
$this->response->setPublicHeaders($this->config->getTtl());
9994
$this->response->setHeader('X-Magento-Tags', implode(',', $this->cacheableQuery->getCacheTags()), true);
@@ -108,6 +103,6 @@ public function afterDispatch(
108103
$this->response->setNoCacheHeaders();
109104
}
110105

111-
return $response;
106+
return $result;
112107
}
113108
}

‎app/code/Magento/GraphQlCache/etc/graphql/di.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
type="Magento\GraphQlCache\Model\Plugin\App\PageCache\Identifier" sortOrder="1"/>
2020
</type>
2121
<type name="Magento\Framework\Controller\ResultInterface">
22+
<plugin name="graphql-result-plugin" type="Magento\GraphQlCache\Controller\Plugin\GraphQl"/>
2223
<plugin name="result-builtin-cache" type="Magento\PageCache\Model\Controller\Result\BuiltinPlugin"/>
2324
<plugin name="result-varnish-cache" type="Magento\PageCache\Model\Controller\Result\VarnishPlugin"/>
2425
</type>

‎dev/tests/integration/testsuite/Magento/GraphQl/Controller/GraphQlControllerTest.php

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -101,11 +101,7 @@ public function testDispatch() : void
101101
$headers = $this->objectManager->create(\Zend\Http\Headers::class)
102102
->addHeaders(['Content-Type' => 'application/json']);
103103
$this->request->setHeaders($headers);
104-
/** @var \Magento\Framework\App\Response\Http $response */
105-
$response = $this->objectManager->create(\Magento\Framework\App\Response\Http::class);
106-
/** @var \Magento\Framework\Controller\Result\Json $result */
107-
$result = $this->graphql->dispatch($this->request);
108-
$result->renderResult($response);
104+
$response = $this->graphql->dispatch($this->request);
109105
$output = $this->jsonSerializer->unserialize($response->getContent());
110106
$linkField = $this->metadataPool->getMetadata(ProductInterface::class)->getLinkField();
111107

@@ -147,11 +143,7 @@ public function testDispatchWithGet() : void
147143
$this->request->setPathInfo('/graphql');
148144
$this->request->setMethod('GET');
149145
$this->request->setQueryValue('query', $query);
150-
/** @var \Magento\Framework\App\Response\Http $response */
151-
$response = $this->objectManager->create(\Magento\Framework\App\Response\Http::class);
152-
/** @var \Magento\Framework\Controller\Result\Json $result */
153-
$result = $this->graphql->dispatch($this->request);
154-
$result->renderResult($response);
146+
$response = $this->graphql->dispatch($this->request);
155147
$output = $this->jsonSerializer->unserialize($response->getContent());
156148
$linkField = $this->metadataPool->getMetadata(ProductInterface::class)->getLinkField();
157149

@@ -203,11 +195,6 @@ public function testDispatchGetWithParameterizedVariables() : void
203195
$this->request->setMethod('GET');
204196
$this->request->setParams($queryParams);
205197
$response = $this->graphql->dispatch($this->request);
206-
/** @var \Magento\Framework\App\Response\Http $response */
207-
$response = $this->objectManager->create(\Magento\Framework\App\Response\Http::class);
208-
/** @var \Magento\Framework\Controller\Result\Json $result */
209-
$result = $this->graphql->dispatch($this->request);
210-
$result->renderResult($response);
211198
$output = $this->jsonSerializer->unserialize($response->getContent());
212199
$linkField = $this->metadataPool->getMetadata(ProductInterface::class)->getLinkField();
213200

@@ -257,11 +244,7 @@ public function testError() : void
257244
$headers = $this->objectManager->create(\Zend\Http\Headers::class)
258245
->addHeaders(['Content-Type' => 'application/json']);
259246
$this->request->setHeaders($headers);
260-
/** @var \Magento\Framework\App\Response\Http $response */
261-
$response = $this->objectManager->create(\Magento\Framework\App\Response\Http::class);
262-
/** @var \Magento\Framework\Controller\Result\Json $result */
263-
$result = $this->graphql->dispatch($this->request);
264-
$result->renderResult($response);
247+
$response = $this->graphql->dispatch($this->request);
265248
$outputResponse = $this->jsonSerializer->unserialize($response->getContent());
266249
if (isset($outputResponse['errors'][0])) {
267250
if (is_array($outputResponse['errors'][0])) {

‎lib/internal/Magento/Framework/App/PageCache/Kernel.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ class Kernel
6969
* @param \Magento\Framework\App\Response\HttpFactory|null $httpFactory
7070
* @param \Magento\Framework\Serialize\SerializerInterface|null $serializer
7171
* @param AppState|null $state
72-
* @param \Magento\PageCache\Model\Cache\Type $fullPageCache
72+
* @param \Magento\PageCache\Model\Cache\Type|null $fullPageCache
7373
*/
7474
public function __construct(
7575
\Magento\Framework\App\PageCache\Cache $cache,

0 commit comments

Comments
 (0)